Skip to content

feat(settings): add mobile Connect this device card - #20982

Open
dschom wants to merge 1 commit into
mainfrom
FXA-14233
Open

feat(settings): add mobile Connect this device card#20982
dschom wants to merge 1 commit into
mainfrom
FXA-14233

Conversation

@dschom

@dschom dschom commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Because

  • We are updating pairing flows and want to land UI components / pages first.
  • Adds the mobile screen asking the user to confirm connecting this device to their account.

This pull request

  • Adds packages/fxa-settings/src/pages/Pair2/Supplicant/ConnectThisDevice/index.tsx.
  • Adds storybook stories for the page.
  • Adds l10n files.

Issue that this pull request solves

Closes: FXA-14233

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).
  • I have manually reviewed all AI generated code.

How to review (Optional)

Check out the storybook output here. Compare against figma designs (linked in ticket). Check out code for AI slop and over all adherence to patterns set forth in FxA.

Note, that this is just UI work, wiring up functionality comes later.

Screenshots (Optional)

See story books.

Other information (Optional)

Follows the pattern set in #20952 (FXA-14234).

@dschom
dschom marked this pull request as ready for review August 10, 2026 18:08
@dschom
dschom requested review from a team as code owners August 10, 2026 18:08
Because:
 - We are updating pairing flows and want to land UI components / pages first.
 - Adds the mobile screen asking the user to confirm connecting this device.

This commit:
 - Adds `packages/fxa-settings/src/pages/Pair2/Supplicant/ConnectThisDevice/index.tsx`.
 - Adds storybook stories for the page.
 - Adds l10n files and tests.

Closes #FXA-14233
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.

2 participants