-
Notifications
You must be signed in to change notification settings - Fork 9
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
Close out the login form once the user has logged in #22
Comments
👋 Hi! This issue has been marked stale due to inactivity. If no further activity occurs, it will automatically be closed in 14 days. |
@upkarlidder I would like to take this issue. |
Sounds good @akshaymattoo. Just confirming with @josiemundi and @sebastiaanzaaijer as I know they were also working on the extension. Thank you! FYI @demilolu |
@upkarlidder Am I good to start on this issue? |
Hi @akshaymattoo, yeah that would be great thanks! |
@sebastiaanzaaijer I have made the changes and tested them as well in my local. It works fine. I have a question I am not able to make another branch out of |
Hello @akshaymattoo, pls create a fork first and then you should be able to push to your fork and create a PR from there. Let us know if that does not work. Thank you again for your contribution! |
@upkarlidder I did open the pull request. Thank you for the direction. |
👋 Hi! This issue has been marked stale due to inactivity. If no further activity occurs, it will automatically be closed in 14 days. |
👋 Hi! This issue has been marked stale due to inactivity. If no further activity occurs, it will automatically be closed in 14 days. |
Is your feature request related to a problem? Please describe.
The extension should close out the login form once the user has logged in. The user could also be given a possibility to log out.
Related to #16
Describe the solution you'd like
Replace the login form with a
logout
button once the user has been authenticated.The text was updated successfully, but these errors were encountered: