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

#166024717 Token verification and expiry for frontend & display user profile info #104

Merged
merged 1 commit into from
May 15, 2019

Conversation

tolulope-od
Copy link
Owner

What does this PR do?

Checks for token expiration for a logged in user
Displays correct user information on the profile card
Adds better loading screen graphic

Description of Task to be completed?

Add verification of token on the frontend to log a user out when the token expires.
Display user's information in the profile page from the information generated from the token

How should this be manually tested?

VIsit the hosted pages here and login as any type of user and visit the profile page.

Any background context you want to provide?

The JWT generated on the server expires after an hour and would not be valid, but it still gets added to requests from the frontend so there has to be a way to clear the token and generate a new one for new requests.

What are the relevant pivotal tracker stories?

#166024717

Screenshots (if appropriate)

Screenshot 2019-05-15 at 10 10 40 AM

Add check for token expiration for a logged in user
Add better loading screen graphic

[Finishes #166024717]
@coveralls
Copy link

Coverage Status

Coverage increased (+0.5%) to 98.182% when pulling 9b0f184 on ch-frontend-token-verification-166024717 into 992b116 on develop.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.727% when pulling 9b0f184 on ch-frontend-token-verification-166024717 into 992b116 on develop.

@tolulope-od tolulope-od merged commit 971d5f7 into develop May 15, 2019
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.

None yet

2 participants