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

Users don't understand how to change Git providers #5975

Closed
chrifro opened this issue Oct 1, 2021 · 8 comments · Fixed by #6484
Closed

Users don't understand how to change Git providers #5975

chrifro opened this issue Oct 1, 2021 · 8 comments · Fixed by #6484
Assignees
Labels
deployed Change is completely running in production needs visual design team: webapp Issue belongs to the WebApp team

Comments

@chrifro
Copy link

chrifro commented Oct 1, 2021

We get several support requests because users don't understand how to change git providers.

Common steps:

  • they try to log in with another Git provider than they used the first time.
  • they get the error "Email address already exists. Log in using a different provider."
  • they are confused and contact support. Some are thinking the solution is deleting the account and creating a new one. So we could assume that some do that without contacting support.

https://gitpod.io/integrations doesn't seem to be obvious

possible solution:

  • adjust error message
  • allow login with another git provider as well (similar to how Vercel is doing it)

cc @gtsiolis

@AlexTugarev
Copy link
Member

allow login with another git provider as well (similar to how Vercel is doing it)

@ChristinFrohne, do you have a story/description for that case?

@svenefftinge svenefftinge added the team: webapp Issue belongs to the WebApp team label Oct 1, 2021
@AlexTugarev
Copy link
Member

@ChristinFrohne, I just checked with vercel.

The thing is, they are storing any Login Connections on backend indefinitely, i.e. even if you disconnect once and try to login, they remember your previous connection and let you in.

@JanKoehnlein
Copy link
Contributor

Sounds like we can only do 1)

@gtsiolis can you provide us with a better error message?

/schedule

@roboquat
Copy link
Contributor

roboquat commented Oct 4, 2021

@JanKoehnlein: Issue scheduled in the meta team (WIP: 0)

In response to this:

Sounds like we can only do 1)

@gtsiolis can you provide us with a better error message?

/schedule

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@JanKoehnlein
Copy link
Contributor

/assign @jldec

@gtsiolis
Copy link
Contributor

Following the comment in #6268 (comment), we could re-use one of the proposed alert titles:

  1. Gitpod account already exists
  2. Selected provider already in use

🍊 🍊 🍊 🍊

Wondering if this is a good time to split sign up and log in flows.

Use case 1️⃣: For example, instead of showing this error message we could ask:

Are you trying to login? It seems there's already an associated account with this email address. Log in

Use case 2️⃣: While if you explicitely visit the sign in page and you don't have an account associated with that email a user could be asked:

There's no existing account with this email address. Need to create an account? Create account

Use case 3️⃣: In the case where you try to select a different provider to create an account we could the user could be asked:

There's an existing Gitpod account using this email address. Try using a different provider and connecting more providers from integrations settings.

@AlexTugarev what do you think?

🍋 🍋 🍋 🍋

Also, @AlexTugarev regarding #5975 (comment), do you think this is something we don't want to aim for? Seems that Vercel bypasses all these issues we have using the approach of storing any login connections on backend indefinitely. ❓

@AlexTugarev
Copy link
Member

I re-read the long story from above and good step forwards boils down to change the message from Email address already exists. Log in using a different provider. to Email address already in use in another account. Please login using github.com.

@AlexTugarev
Copy link
Member

Also, @AlexTugarev regarding #5975 (comment), do you think this is something we don't want to aim for? Seems that Vercel bypasses all these issues we have using the approach of storing any login connections on backend indefinitely. ❓

@gtsiolis, while this definitely sounds like it can improve the UX, the downside is that we store account connection indefinitely. Do we want to do this?

@AlexTugarev AlexTugarev self-assigned this Nov 1, 2021
AlexTugarev added a commit that referenced this issue Nov 1, 2021
1. Advertise to use another identity provider.
2. Handle same error in New Project wizard.

Fixes #5975
AlexTugarev added a commit that referenced this issue Nov 1, 2021
1. Advertise to use another identity provider.
2. Handle same error in New Project wizard.

Fixes #5975
AlexTugarev added a commit that referenced this issue Nov 1, 2021
1. Advertise to use another identity provider.
2. Handle same error in New Project wizard.

Fixes #5975
roboquat pushed a commit that referenced this issue Nov 2, 2021
1. Advertise to use another identity provider.
2. Handle same error in New Project wizard.

Fixes #5975
@jldec jldec added deployed Change is completely running in production and removed groundwork: awaiting deployment labels Nov 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed Change is completely running in production needs visual design team: webapp Issue belongs to the WebApp team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants