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

Profile Page UI #69

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Profile Page UI #69

wants to merge 2 commits into from

Conversation

lamaro02
Copy link
Contributor

@lamaro02 lamaro02 commented Jun 7, 2024

Developer: Lindsey Amaro

Closes #49 (note: this doesn't close the entire issue--there are still things to be done that I did not get the chance to finish, but I wanted to open this pull request for future developers to see the progress)

Pull Request Summary

I created the user interface for the profile page displaying the currently logged in user's name and email. I also added routing in the navigation bar to this profile page. To be completed, the part of this issue I was unable to complete is connecting the profile page to the application's back end in order to find and display the currently logged-in user's information. I tried using JWT web tokens and React Context API in order to so, but was unsuccessful.

Modifications

Navbar.tsx - added routing to profile page
login/page.tsx - To my knowledge and testing, everything about the login page still works correctly, I just changed the routing after a user logs in to bring them to the homePage instead of clientPage, but that can be disregarded.
profilePage.tsx and profilePage.module.css - UI for the profile page

Testing Considerations

Tested the login and navigation to the profile page on my machine, and everything worked properly.

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

image

@lamaro02 lamaro02 requested a review from hopieyimmie June 7, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Profile Page & Logout implementation
1 participant