Skip to content

Conversation

@dschom
Copy link
Contributor

@dschom dschom commented Jun 4, 2025

Because

  • Check authenticated is a better option when a route requires a sessionToken

This pull request

  • Switches over to using checkAuthenticated
  • Adds email as a parameter to the checkAuthenticated call, which allows us to use emails filters

Issue that this pull request solves

Closes: FXA-11777

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.

@dschom dschom requested a review from a team as a code owner June 4, 2025 23:28
@dschom dschom force-pushed the use-check-authenticated branch from 0a1c1bc to c2b1a6c Compare June 5, 2025 19:29
Because:
- Check authenticated is a better option when a route requires a sessionToken

This Commit:
- Switches over to using checkAuthenticated
@dschom dschom force-pushed the use-check-authenticated branch from c2b1a6c to 5dd2a66 Compare June 5, 2025 20:48
@dschom dschom merged commit 597476a into main Jun 6, 2025
19 checks passed
@dschom dschom deleted the use-check-authenticated branch June 6, 2025 18:26
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