Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

BurgerMenu on mobile view could benefit from a logout option #281

Open
johngribbin opened this issue Apr 1, 2020 · 2 comments
Open

BurgerMenu on mobile view could benefit from a logout option #281

johngribbin opened this issue Apr 1, 2020 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@johngribbin
Copy link
Collaborator

Screen Shot 2020-04-01 at 12 39 24 PM

  • This should be a quick fix, and would save the user from visiting their settings page and logging out from there
  • The function to log a user out can be taken from the AccountSettings component and reused for this:
  const logout = () => {
    localStorage.removeItem("trusat-login-credentials");
    localStorage.removeItem("trusat-allow-cookies");
    history.push("/");
    window.location.reload();
  };
@johngribbin johngribbin added enhancement New feature or request good first issue Good for newcomers labels Apr 1, 2020
@martinvarelaaaa
Copy link

@johngribbin I would like to start collaborating on this issue! How about?

@johngribbin
Copy link
Collaborator Author

johngribbin commented May 27, 2020

@martinvarelaaaa Go for it Martín! Cheers :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants