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

Allow users to unlink GitHub accounts #1622

Closed
outoftime opened this issue Dec 11, 2018 · 1 comment
Closed

Allow users to unlink GitHub accounts #1622

outoftime opened this issue Dec 11, 2018 · 1 comment
Assignees

Comments

@outoftime
Copy link
Contributor

Since GitHub login sessions are indefinitely long-lived, it’s common for a student to be on a shared laptop that’s logged in to a different student’s GitHub account. Thus, it’s a common problem for a student to accidentally link a different student’s GitHub account to their own Popcode account. While the “merge accounts” flow includes an explicit confirmation that we’re looking at the right GitHub account, if the erroneous GitHub account is not yet associated with a Popcode account, there is no such confirmation.

So, we should allow students to unlink the GitHub account that‘s linked to their Popcode account, to make it easy to undo this mistake.

From a UX perspective, we will want to show enough information about the linked GitHub account to allow the student to make an informed decision about whether it’s the correct one. So, add a component to the logged-in user drop-down that shows the user’s GitHub avatar, display name, and username, along with an Unlink button. For consistency, we’ll just show the same information about the linked Google account, although there will be no Unlink option here because Google is the primary authentication provider for Popcode.

@outoftime outoftime self-assigned this Dec 11, 2018
@outoftime
Copy link
Contributor Author

outoftime commented Feb 20, 2019

Closed by #1637

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant