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

Add client query to fetch user profile #149

Closed
3 tasks done
minhhang107 opened this issue Feb 7, 2022 · 0 comments · Fixed by #151
Closed
3 tasks done

Add client query to fetch user profile #149

minhhang107 opened this issue Feb 7, 2022 · 0 comments · Fixed by #151
Assignees
Labels
feature Feature request frontend priority:medium Issue to be done in the current sprint
Milestone

Comments

@minhhang107
Copy link
Collaborator

minhhang107 commented Feb 7, 2022

Is your feature request related to a problem? Please describe.
Currently the data on Profile page is hard coded. We need to add query to fetch data from database

Are there any related issues? Please list all of them.
#34

Describe the solution you'd like

  • add query getUserProfileById
  • implement state management
  • implement a way to display Edit profile correctly

Acceptance criteria

  • User can view the correct information on profile page
  • User can view button Edit profile only when they're on their own profile page

Describe alternatives you've considered
N/A

Additional context
May use jwt-decode to decode the token stored in localStorage to get the current logged-in user

@minhhang107 minhhang107 added frontend feature Feature request priority:medium Issue to be done in the current sprint labels Feb 7, 2022
@minhhang107 minhhang107 added this to the Milestone 1 milestone Feb 7, 2022
@minhhang107 minhhang107 self-assigned this Feb 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature request frontend priority:medium Issue to be done in the current sprint
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant