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.

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

Create endpoint for Adding Buddy #28

Closed
shreyasingaraju opened this issue Mar 12, 2022 · 3 comments · Fixed by #37
Closed

Create endpoint for Adding Buddy #28

shreyasingaraju opened this issue Mar 12, 2022 · 3 comments · Fixed by #37
Assignees
Labels
Priority: Medium Status: Review Needed The implementation has finished and someone has been approached for review. Type: Feature The issue is about a new feature being introduced to the project.

Comments

@shreyasingaraju
Copy link
Contributor

shreyasingaraju commented Mar 12, 2022

Describe the task that needs to be done.
Set up endpoint and documentation for adding a buddy by communicating with 'Users'.

Describe how a solution to your proposed task might look like (and any alternatives considered).
Set up empty endpoint methods with descriptions for adding a buddy that will show up on swagger.

Endpoint: addBuddy
Path: /api/pairing/
Request: Current User ID and added buddy's User ID.
Operation: PUT
Response: 200 OK. Adds new buddy's User ID to current user's array of all buddies.

Notes

@shreyasingaraju shreyasingaraju added Priority: Medium Status: In Progress The issue is currently been worked on by someone. Type: Documentation The issue is about adding or modifying the existing documentation for the project. labels Mar 12, 2022
@f-fromont f-fromont added Status: Review Needed The implementation has finished and someone has been approached for review. and removed Status: In Progress The issue is currently been worked on by someone. labels Mar 12, 2022
@f-fromont
Copy link
Contributor

f-fromont commented Mar 12, 2022

Would you be able to include examples of the endpoint details in the solution description?

For example,

Endpoint: addBuddy
path: /api/pairing/
Request: Contains array of course Id's
Response: 200 OK

If you have any questions about what the endpoint should look like for now, don't hesitate to reach out here or on the discord :)

@f-fromont
Copy link
Contributor

LGTM with the additions, assigned to self to get working on it!

@f-fromont f-fromont added Status: In Progress The issue is currently been worked on by someone. and removed Status: Review Needed The implementation has finished and someone has been approached for review. labels Mar 13, 2022
@f-fromont f-fromont added Status: Review Needed The implementation has finished and someone has been approached for review. Type: Feature The issue is about a new feature being introduced to the project. and removed Status: In Progress The issue is currently been worked on by someone. Type: Documentation The issue is about adding or modifying the existing documentation for the project. labels Mar 14, 2022
@f-fromont
Copy link
Contributor

This issue is blocking #47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium Status: Review Needed The implementation has finished and someone has been approached for review. Type: Feature The issue is about a new feature being introduced to the project.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants