Skip to content

fix(bug): Remove email from query param when clicking Change email #19134

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vbudhram
Copy link
Contributor

@vbudhram vbudhram commented Jul 3, 2025

Because

  • When users change their email address during sign-in or sign-up, the application should clear any prefilled email from the query parameters to avoid confusion and ensure the correct email is used.

This pull request

  • Updates the sign-in and sign-up flows to remove the email query parameter from the URL when users choose to change their email address.
  • Ensures that the navigation back to the entry page (/) includes any remaining query parameters except for email.
  • Updates related tests to expect the new navigation behavior.

Issue that this pull request solves

Closes: https://mozilla-hub.atlassian.net/browse/FXA-11938
Closes: https://mozilla-hub.atlassian.net/browse/FXA-12010

Checklist

  • 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)

@vbudhram vbudhram requested a review from a team as a code owner July 3, 2025 13:20
@vbudhram vbudhram self-assigned this Jul 3, 2025
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.

1 participant