-
Notifications
You must be signed in to change notification settings - Fork 8
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
#167164944 View an Edit Author Profile #30
base: develop
Are you sure you want to change the base?
Conversation
70b8343
to
7bfe4dc
Compare
- design and create components for viewing profile - add unit testing [Starts #167164944]
7bfe4dc
to
89bb5a7
Compare
89bb5a7
to
64220bb
Compare
2fe1c99
to
4f421e2
Compare
const { token } = JSON.parse(window.localStorage.getItem('AuthorsHavenUser')) || { token: null }; | ||
if (token) { | ||
|
||
const user = JSON.parse(window.localStorage.getItem('AuthorsHavenUser')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is to handle a case where the AuthorsHavenUser is deleted from the local storage when still logged in (i.e not redirected to the landing page). I faced error when that happened.
- handle state updates properly to avoid window reload - add and link "START WRITING"
What does this PR do?
Description of Task to be completed?
/profile
e.g http://localhost:3000/profile
How should this be manually tested?
To test manually on your local computer with
localhost
.env.example
file on the environmental variables to provideNote: Kindly checkouts the screenshots section the layout to expect
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#167164944
Screenshots