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

Update docs to account for unconfirmed users #775

Merged

Conversation

dan-klasson
Copy link
Contributor

@dan-klasson dan-klasson commented Aug 19, 2024

Ensure oauth users cannot login with unconfirmed account #443

Ensure oauth users cannot login with unconfirmed account team-alembic#443
@dan-klasson dan-klasson force-pushed the update-oauth-docs-uncomfired branch 4 times, most recently from 05531c8 to 038eecf Compare August 19, 2024 16:16
@zachdaniel
Copy link
Collaborator

I think we want to add some comments to highlight that these changes are only necessary if you're using the password strategy & confirmation.

@dan-klasson dan-klasson force-pushed the update-oauth-docs-uncomfired branch 4 times, most recently from 51131fc to d25900e Compare August 19, 2024 16:24
@zachdaniel
Copy link
Collaborator

I'll let @jimsynz review, check the discord conversation for more on the issue that this addresses when combining social logins with email/password. https://discord.com/channels/711271361523351632/1275053139527204874

@jimsynz jimsynz merged commit baade96 into team-alembic:main Sep 1, 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.

3 participants