Skip to content

ref(settings): migrate account emails to new form system#110872

Merged
natemoo-re merged 3 commits intomasterfrom
nm/form/DE-953
Mar 18, 2026
Merged

ref(settings): migrate account emails to new form system#110872
natemoo-re merged 3 commits intomasterfrom
nm/form/DE-953

Conversation

@natemoo-re
Copy link
Member

@natemoo-re natemoo-re commented Mar 17, 2026

Migrates the /settings/account/notifications/email pages to new form system.

  • Replaces styled components with core layout components
  • Replaces useLocation with useQueryState
  • Removes dead code (AccountNotificationsByOrganization is not accessible)

Barely any visual diff, big code cleanup.

Before After
email-before email-after

@natemoo-re natemoo-re requested a review from a team March 17, 2026 16:59
@linear-code
Copy link

linear-code bot commented Mar 17, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 17, 2026
Copy link
Collaborator

@TkDodo TkDodo left a comment

Choose a reason for hiding this comment

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

wow, this is great 🔥

@natemoo-re natemoo-re enabled auto-merge (squash) March 18, 2026 16:15
@natemoo-re natemoo-re merged commit f395247 into master Mar 18, 2026
63 checks passed
@natemoo-re natemoo-re deleted the nm/form/DE-953 branch March 18, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants