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

Migrate the frontend to the new GET /user/me endpoint #906

Open
4 tasks
eleanorreem opened this issue May 15, 2024 · 3 comments
Open
4 tasks

Migrate the frontend to the new GET /user/me endpoint #906

eleanorreem opened this issue May 15, 2024 · 3 comments
Assignees
@LouisVdwl
Labels
complexity: beginner Time needed to do this ticket is less than 1 day. maintenance Maintenance / chore work priority: 1+week Should be prioritized next week or longer. state: approved Ready to go. Not blocked or pending.
Milestone
04. Maintenance a...

Comments

@eleanorreem
Copy link
Contributor

eleanorreem commented May 15, 2024 •
edited

Overview

We are not very compliant with REST standards. We would like to be better. This ticket focuses on the current POST /user/me endpoint. This endpoint GETs information and is not a POST request.

Action Items

  • Look at recent GET /user/me endpoint
  • Update the current implementation of the POST /user/me endpoint to a GET request in /bloom-frontend/app/api.ts
  • Ensure there are no other places where this api endpoint is used
  • Run Cypress tests and post a picture of them passing. This is super important as this change will touch most functionality.
@eleanorreem eleanorreem added complexity: beginner Time needed to do this ticket is less than 1 day. maintenance Maintenance / chore work state: approved Ready to go. Not blocked or pending. priority: 1+week Should be prioritized next week or longer. labels May 15, 2024
@eleanorreem eleanorreem mentioned this issue May 15, 2024
Open
3 tasks
@LouisVdwl
Copy link

Hey @eleanorreem ! I can work on it :)

@github-actions GitHub Actions
Copy link
Contributor

Thank you for your interest in contributing to Chayn! Please carefully read the CONTRIBUTING.md file and the README.md file for guidance. Let us know if you have any questions. Good luck!

@eleanorreem
Copy link
Contributor Author

Hey @LouisVdwl, Thanks for your interest. Let me know if you have any questions!

@SangilYun SangilYun mentioned this issue May 19, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LouisVdwl LouisVdwl

Labels
complexity: beginner Time needed to do this ticket is less than 1 day. maintenance Maintenance / chore work priority: 1+week Should be prioritized next week or longer. state: approved Ready to go. Not blocked or pending.
Projects
None yet
Milestone
04. Maintenance and Testing
Development

No branches or pull requests
3 participants
@eleanorreem @kyleecodes @LouisVdwl