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

Prompt for Google account selection every time on log in. #400

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mrabbitt
Copy link
Contributor

@mrabbitt mrabbitt commented Aug 30, 2024

(I think this change is probably a good idea, but would like to hear what others think, so I've left it as a draft PR for now.)

Currently, when you logout of Compass, and then sign in again with the "Sign in with Google" buton on the Sign In page, the Students page is loaded immediately without prompting for confirmation or selection of which Google account to use.

I figured out how to force the Google NextAuth provider to prompt for Google account selection and confirmation every time the "Sign in with Google" button is clicked. Note that if a user never logs out of Compass or clears their auth cookies, they will still stay signed in.

For what it's worth, I checked the behavior of the Logout link in Figma using Google SSO, and it also prompts for account selection and confirmation when logging back in afterward.

Before this change:

Screen.Recording.2024-08-30.at.3.46.23.PM.mov

After this change:

Screen.Recording.2024-08-30.at.3.45.29.PM.mov

@thomhickey
Copy link
Contributor

Nice find @mrabbitt, but it doesn't really prompt for sign-in it just prompts for account selection. I am not challenged for my password.

@mrabbitt
Copy link
Contributor Author

mrabbitt commented Aug 31, 2024

Nice find @mrabbitt, but it doesn't really prompt for sign-in it just prompts for account selection. I am not challenged for my password.

@thomhickey This change is not intended to address your "issue" that it doesn't ask for a password. I don't think it's clear that this should be a requirement of Compass. I think we should wait for clarification or firm requirements from our PM and/or SFUSD. Thanks.

@thomhickey
Copy link
Contributor

Right on, just wanted to be clear because your PR title says prompt for google sign-in.

@mrabbitt mrabbitt changed the title Prompt for Google sign-in every time on log in. Prompt for Google account selection every time on log in. Aug 31, 2024
@mrabbitt
Copy link
Contributor Author

PR title updated to clarify scope.

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

Successfully merging this pull request may close these issues.

2 participants