-
Notifications
You must be signed in to change notification settings - Fork 54
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
UI needs to implement a logout page/button/route #78
Comments
I don't think we need a new page. Just a logout button should suffice. An even quicker fix could be to just implement a " |
@Zachary-Fernandes -- See #41 for information about how the |
@yhwang -- could you provide some guidance, as to how a logout function should be implemented? ... removing session cookies, |
@ckadner yes, a route to handle /logout should be enough. but we also need to handle the /logout on the server side to remove the session. |
I can also work on this issue as well if zack hasn’t started on it, or needs some help |
Is your feature request related to a problem? Please describe.
Right now there's no way for the admin to logout other than clear up the browser cookies, so we need a logout page to simply this process.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: