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

Added Logout Functionality - #300

Open
criticic wants to merge 1 commit into
JBossOutreachArchive:masterfrom
criticic:addLogoutFunctionality
Open

Added Logout Functionality#300
criticic wants to merge 1 commit into
JBossOutreachArchive:masterfrom
criticic:addLogoutFunctionality

Conversation

@criticic

Copy link
Copy Markdown

This mainly works by removing the login token from LocalStorage

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

  • This Pull request
    • Implements a new feature.
  • I have read the [CONTRIBUTING.md] .
  • I have included a description of changes below.
  • I have squashed the commits (mandatory).

Description / Changes

@criticic
criticic force-pushed the addLogoutFunctionality branch from 42cdf9b to aef1477 Compare December 22, 2019 10:57
@criticic

Copy link
Copy Markdown
Author

I took help from a stackoverflow answer on how to remove localstorage items. Does that count as Plagiarism?

@anshumanv

Copy link
Copy Markdown

I took help from a stackoverflow answer on how to remove localstorage items. Does that count as Plagiarism?

Nope, that's fine. 👍

@anshumanv anshumanv left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@anshumanv

Copy link
Copy Markdown

@KeenWarrior can you trigger the build again? Seems to have stopped abruptly.

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