-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Catch all errors from sign in - improvement for signInFailure #1074
Comments
i can fix this @Boardtale |
Hi @Boardtale, thanks for reporting this. I agree that As a side note, would email link authentication be a better fit for your use case than email + password? |
What are your difficulties
problem
…On Thursday, March 7, 2024, Thanush19 ***@***.***> wrote:
i can fix this @Boardtale <https://github.com/Boardtale>
can i work on this??
—
Reply to this email directly, view it on GitHub
<#1074 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A2WGGN7R5I7VEESS6OKV6TLYW7VZDAVCNFSM6AAAAABED3R3ECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBSGMZTKNJWHA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Glad that I can confirm that I am not the only one experiencing this. The only potential approach for me now is to use firebase SDK to initiate sign in flow which then defeats the purpose of me using this library. Edit: |
Need:
I require for email + pwd to verify emails.
As far as I know, I can't prevent or redirect flow after user first time creates account. So the error is sent from my blocking function: 'please verify email'.
That does not look good when user tries to log in with unverfied email:
I want to catch that error and show proper error popup for the users.
I am using 'signInFailure' but it's not triggered.. I think ever :P I don't know where it's triggered but not if I get any forbidden from my blocking function
So I'd like to solve it somehow. I see few paths but neither is possible - afaik - with firebaseUI. Unless I'm gonna dirty trick it and break encapsulation.
EDIT:
I guess I could do this manually on my own backend, after login, check if verified email and then redirect them also logging them out. But that's many additional actions
The text was updated successfully, but these errors were encountered: