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

Add a "Recent Logins" list to the configuration page #49

Closed
fmitchell-r7 opened this issue Apr 7, 2016 · 4 comments
Closed

Add a "Recent Logins" list to the configuration page #49

fmitchell-r7 opened this issue Apr 7, 2016 · 4 comments

Comments

@fmitchell-r7
Copy link
Contributor

To facilitate account switching, users need a way to log out (see #48) and a way to log back in. The current "remember the last metadata URL used" method isn't sufficient for handling multiple accounts. We should add a "Recent Logins" list to the configuration page. The list should be of the last six URLs successfully used to login.

If a URL in the "SAML Metadata URL" box is already in the list, the existing URL in the list should be moved to the top and the list length shouldn't change. If a URL in the box isn't already in the list, the URL should be added to the top of the list and the list truncated to the last six used URLs.

URLs should be ordered on the configuration page with the most recently used URL at the top of the list. URLs in the list should be selectable and copyable so that users can cut and paste them into the "SAML Metadata URL" box.

The maximum length of the list should be stored as a configuration parameter.

@rhodgman-r7
Copy link

This would be a great feature to have. You might even consider enhancing this:

URLs should be ordered on the configuration page with the most recently used URL
at the top of the list. URLs in the list should be selectable and copyable so that
users can cut and paste them into the "SAML Metadata URL" box.

by putting a button next to each of the recent urls which would allow the user to click that button and initiate the login sequence instead of having to copy and paste the url.

@erran-r7
Copy link
Contributor

@asharma-r7 this is the issue I mentioned that we'd love some UI ✨ magic on. 😉

erran-r7 pushed a commit that referenced this issue Oct 5, 2016
🎉 Use clipboard.js to support pastable 🍝 URL goodness.

Additionally these changes plan for supporting #65 in the future by
supporting a mapping of metadata URL to profile names.
@rhodgman-r7
Copy link

I'm psyched that you're working on this, I switch accounts pretty frequently and this will make things easier. Thank you.

@erran-r7
Copy link
Contributor

erran-r7 commented Oct 5, 2016

@rhodgman-r7 👍 I'm excited. Currently I've implemented the copy/paste format. Eventually we may make the button a "log into existing provider" button instead (it currently copies to the clipboard so only requires paste/click).

erran-r7 added a commit that referenced this issue Oct 13, 2016
…rable

Fix #49 - Implement a "recent logins" list on the Configure page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

3 participants