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

Fix sign out of oauth provider #111

Closed

Conversation

FatumaA
Copy link
Collaborator

@FatumaA FatumaA commented Sep 6, 2024

What kind of change does this PR introduce?

Logs out the google provider

What is the current behavior?

Closes #78

What is the new behavior?

Logs out of the google provider and the supabase session, making the chooser appear whenever the user tries to log back in.

@FatumaA FatumaA requested a review from dshukertjr September 6, 2024 13:09
Copy link
Member

@dshukertjr dshukertjr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, now that I think about it after seeing this PR, it might be better to leave it up to the developer to sign out from Google accounts. I personally don't think introducing a new sign-out method is a good developer experience upgrade.

Maybe we can just add comments on the third-party sign-in methods that singing out from the third party auth provider should be handled by the developer and call it a day?

@FatumaA
Copy link
Collaborator Author

FatumaA commented Sep 7, 2024

We can do that too.
Although I think even with this signout introduction, developers can still choose whether or not to use it

@dshukertjr
Copy link
Member

I'm just not a fan of having two separate sign out methods. It's very counter intuitive, and feel like would cause confusion. There is also the issue of discoverability. I think the way to solve this issue is with good documentation.

@FatumaA
Copy link
Collaborator Author

FatumaA commented Sep 9, 2024

Agreed

@FatumaA FatumaA closed this Sep 9, 2024
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.

If signed in with Google provider, signOut does not sign out of the provider
2 participants