Skip to content

feat(settings): Add mapping for auth passkey errors#20056

Merged
nshirley merged 1 commit intomainfrom
FXA-12910
Feb 25, 2026
Merged

feat(settings): Add mapping for auth passkey errors#20056
nshirley merged 1 commit intomainfrom
FXA-12910

Conversation

@nshirley
Copy link
Copy Markdown
Contributor

Because:

  • We want consistent error numbers between backend auth and settings

This Commit:

  • Adds new settings auth errors using previously defined passkey errno

Closes: FXA-12910

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

Because:
 - We want consistent error numbers between backend auth and settings

This Commit:
 - Adds new settings auth errors using previously defined passkey errno

Closes: FXA-12910
@nshirley nshirley requested review from a team as code owners February 12, 2026 23:44
auth-error-216 = Text message limit reached
auth-error-218 = Unable to remove recovery phone, missing backup authentication codes.
auth-error-219 = This phone number has been registered with too many accounts. Please try a different number.
auth-error-224 = Passkey not found
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I just assumed these would match the underlying error message

@nshirley nshirley merged commit 4115a9c into main Feb 25, 2026
22 checks passed
@nshirley nshirley deleted the FXA-12910 branch February 25, 2026 18:03
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.

4 participants