Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

10 implement feed #14

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

10 implement feed #14

wants to merge 21 commits into from

Conversation

Saverio976
Copy link

@Saverio976 Saverio976 commented Jul 19, 2023 •
edited
Loading

Description

command feed:

  • create
  • add
  • del
  • drop

#10

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

  • I have assigned this PR to myself
  • I have added at least 1 reviewer
  • I have added the needed labels
  • I have linked this PR to an issue
  • I have linked this PR to a milestone
  • I have linked this PR to a project
  • I have tested this code
  • I have added / updated tests (unit / functional / end-to-end / ...)
  • I have updated the README and other relevant documents (guides...)
  • I have added sufficient documentation both in code, as well as in the READMEs

@Saverio976 Saverio976 added the enhancement New feature or request label Jul 19, 2023
Saverio976
Saverio976 commented Jul 22, 2023
View reviewed changes
src/commands/feed/info.ts Outdated Show resolved Hide resolved
src/commands/feed/info.ts Show resolved Hide resolved
@Saverio976 Saverio976 marked this pull request as ready for review August 10, 2023 06:35
adamdeziri
adamdeziri suggested changes Aug 23, 2023
View reviewed changes
Copy link

@adamdeziri adamdeziri left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

U the readme to include the use of the feed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamdeziri adamdeziri adamdeziri approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

@Saverio976 Saverio976

@YlanGarnier YlanGarnier

Labels
enhancement New feature or request
Projects
None yet
Milestone
Sprint 3 - new commands and UX improv...
Development

Successfully merging this pull request may close these issues.
3 participants
@Saverio976 @adamdeziri @YlanGarnier