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

Fix navbar username change after updateUser #109

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

Conversation

agt-ru
Copy link

@agt-ru agt-ru commented Feb 18, 2021

This little fix makes the navbar username change after a user edits their profile in Admin - Users.

@agt-ru
Copy link
Author

agt-ru commented Feb 24, 2021

I've added another commit to update the local storage after successful login for the fix to work properly.

@talmax1124
Copy link

This fix already has been implemented In the last episode of the udemy course. Please close this PR

@agt-ru
Copy link
Author

agt-ru commented Mar 25, 2021

This fix already has been implemented In the last episode of the udemy course. Please close this PR

Carlos, you mean, it's implemented only in course, not here in the repository?
Have you fixed it in your repo? Could you link to your commit, please?

@talmax1124
Copy link

@agt-ru It already has been implemented in this commit: 4eca6ce

@talmax1124
Copy link

@agt-ru Can you help me integrate a feature? I'm new to react, and need help integrating this

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.

2 participants