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 a new type called UserSession and use it in the UserNav component #41

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

jhodapp
Copy link
Member

@jhodapp jhodapp commented Oct 10, 2024

Description

This PR adds a new type called UserSession and uses it in the UserNav component.

GitHub Issue: None

Changes

  • Add a new type called UserSession which holds more than just userId
  • Use the new fields from the AuthStore in the UserNav component

Screenshots / Videos Showing UI Changes (if applicable)

Refactor Coaching _ Mentoring

Testing Strategy

  1. Log in
  2. View the UserNav menu (upper right hand corner menu)
  3. Notice that the 2 letter icon is dynamically set to "FL" where FL == [F]irst + [L]ast
  4. Notice that the user's full name is dynamically set
  5. Notice that the user's email address is dynamically set

Concerns

None

@jhodapp jhodapp added enhancement Improves existing functionality or feature feature work Specifically implementing a new feature labels Oct 10, 2024
@jhodapp jhodapp self-assigned this Oct 10, 2024
Copy link
Member Author

@jhodapp jhodapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-approving while David's fully occupied this week.

@jhodapp jhodapp merged commit 9321f56 into main Oct 10, 2024
1 check passed
@jhodapp jhodapp deleted the dynamic_user_nav_menu branch October 10, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves existing functionality or feature feature work Specifically implementing a new feature
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

1 participant