Repository navigation
AUTH-MAIL-002A: report verification-email request outcome truthfully - #148
Merged
Merged
Conversation
✅ Deploy Preview for luminous-fox-7c393f ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This was referenced Jul 13, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #145
Outcome
Registration now reports two separate facts: whether the account was created and whether Firebase accepted the verification-email request. If the second step fails, the signed-in account is preserved, no provider detail is exposed, and the member receives one safe inspection/stop path instead of a false Inbox claim.
State transitions
Safety and compatibility
email_verification_failed.Verification
git diff --check: passed.Officer handoff
Officer impact: Members see account creation and email-request acceptance separately. If the request fails, they keep the account and receive a safe stop/escalation message.
Officer documentation:
docs/officers/EVENTS_SHOP_MEMBERS.md;docs/officers/EMERGENCY_AND_RECOVERY.md.Deployment evidence: Source commit
7c66602and local tests are proven. Code merge, website publication,runmprc.com, Firebase deployment, provider configuration, and production behavior are not yet proven. No production email was sent.Rollback
Revert this pull request. There is no data migration or provider configuration to undo.
Post-merge release result
46557c7988a496c1081e268baad6f33fb186ee0f.29247002544: passed.29247125844: exact-main/CI preflight passed, then failed closed because required public App Check build configuration is absent.runmprc.com: HTTP 200 on 2026-07-13, but still served prior assetmain.58ede287.js; the new registration copy is not live.