-
Notifications
You must be signed in to change notification settings - Fork 0
Get all projects with GET /projects
#70
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.
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
Conversation
nourshoreibah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM great work!!
nourshoreibah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updating to match users (the "start-server-and-test" stuff is in lambda cli now for future)
Co-authored-by: Nour Shoreibah <[email protected]>
Co-authored-by: Nour Shoreibah <[email protected]>
|
Good work!! |
Yurika-Kan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
everything looks good! splendid might I say
nourshoreibah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the comment on the swagger/openapi file, looks great!
ℹ️ Issue
Closes #59
📝 Description
you can now get a json response of all the projects in the database by hitting this endpoint
✔️ Verification
wrote tests
🏕️ (Optional) Future Work / Notes
maybe pagination, nour said not required