Skip to content
This repository was archived by the owner on Apr 12, 2023. It is now read-only.

Logout button - #307

Open
snowy-night-light wants to merge 1 commit into
JBossOutreachArchive:masterfrom
snowy-night-light:Logout
Open

Logout button#307
snowy-night-light wants to merge 1 commit into
JBossOutreachArchive:masterfrom
snowy-night-light:Logout

Conversation

@snowy-night-light

Copy link
Copy Markdown

Check by changing each [ ] to [x] Please take note of the white-space as it matters.

  • This Pull request
    • Fixes a bug.
    • Implements a new feature.
  • I have read the [CONTRIBUTING.md] .
  • I have included a description of changes below.
  • I have included screenshots showing before and after the changes were made.
  • I have squashed the commits (mandatory).

Description / Changes

  • Added logout button in navbar

Screenshots

https://www.youtube.com/watch?v=26YFDIryDws

return localStorage.getItem('token') != null;
}

logout() {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You can use router's API to redirect to the index/signin page rather than relying on browser's href.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants