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

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 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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