Dashboard: warn when Profile email uses a custom domain by StevenDufresne · Pull Request #112201 · Automattic/wp-calypso

StevenDufresne · 2026-07-02T05:22:58Z

Proposed Changes

Add a warning under the Email address field on the Dashboard Profile page when the account email is on a custom domain (any domain not on a known free-provider list).
The warning explains that account recovery would be lost if the domain expires, and suggests a free email provider.
Ports the behavior from Calypso's account settings. The Dashboard is self-contained (it doesn't reuse Calypso client code), so the free-provider detection is reimplemented locally.

Current	This PR

Why are these changes being made?

The new Dashboard Profile page dropped the custom-domain email warning that Calypso shows. Because a user's account recovery is tied to that email, losing the domain can lock them out — so the warning should carry over.

Considerations

Use a banner

This looks better, but demands visual attention.

Testing Instructions

Go to /me/account on the Dashboard.
With the account email on a custom domain (or type e.g. test@somedomain.com), confirm the warning appears under the Email address field.
With a free provider (e.g. test@gmail.com), confirm no warning shows.
Confirm the warning is hidden while an email change is pending.

Considerations

Local vs shared helper: duplicated the free-provider allowlist locally rather than extracting a shared package, to respect the Dashboard's self-contained architecture. The list changes rarely, so the duplication is cheap.
Inline text vs banner: presented as help text under the input (matching Calypso's altitude) rather than a Notice banner.

Pre-merge Checklist

🤖 Generated with Claude Code

Ports the custom-domain account-recovery warning from Calypso's account settings to the new Dashboard Profile page. Ref: DOTMSD-1310

Render the warning as help text under the email input (matching Calypso's altitude) instead of a Notice banner. Ref: DOTMSD-1310

Drop the custom has-warning styling; show the warning through the InputControl help prop with default styling. No bespoke CSS. Ref: DOTMSD-1310

Ref: DOTMSD-1310

github-actions · 2026-07-02T05:23:07Z

Calypso Live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-186174

Jetpack Cloud Live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-186174&env=jetpack

Automattic for Agencies Live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-186174&env=a8c-for-agencies

Dashboard Live (dotcom) (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-186174&env=dashboard

Dashboard Live (A4A) (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-186174&env=dashboard-a4a

…tion test Per client/dashboard/docs/testing.md, simple utilities are covered via integration tests. Move the subdomain edge case into index.test.tsx. Ref: DOTMSD-1310

Ref: DOTMSD-1310

fushar

Tested. Works. Left a very small comment.

BTW it seems the before-after images in the PR description are the same.

Co-authored-by: Ashar Fuadi <ashar.fuadi@automattic.com>

StevenDufresne · 2026-07-02T22:57:31Z

BTW it seems the before-after images in the PR description are the same.

They are not. See label case :). That's how great this PR is.

github-actions · 2026-07-02T23:06:49Z

Looks like one of the E2E tests has failed.

You can fix them following these steps:

Check out this branch locally:
```
gh pr checkout 112201
```
Start Claude Code in the repo:
```
claude
```
Run the /fix-e2e-tests skill, passing this PR number:
```
/fix-e2e-tests 112201
```

fushar

They are not. See label case :). That's how great this PR is.

Ohhhh I see, I thought Current means the MSD profile page before this PR 👍

StevenDufresne · 2026-07-03T02:28:05Z

Ohhhh I see, I thought Current means the MSD profile page before this PR

No, :). Current means calypso's account page. Confusion noted. Will be more accurate with copy moving forward.

a8ci18n · 2026-07-06T01:22:35Z

This Pull Request is now available for translation here: https://translate.wordpress.com/deliverables/34220110

Some locales (Hebrew) have been temporarily machine-translated due to translator availability. All other translations are usually ready within a few days. Untranslated and machine-translated strings will be sent for translation next Monday and are expected to be completed by the following Friday.

Thank you @StevenDufresne for including a screenshot in the description! This is really helpful for our translators.

a8ci18n · 2026-07-08T19:28:47Z

Translation for this Pull Request has now been finished.

StevenDufresne added 4 commits July 2, 2026 12:55

Dashboard: warn when Profile email uses a custom domain

8457b52

Ports the custom-domain account-recovery warning from Calypso's account settings to the new Dashboard Profile page. Ref: DOTMSD-1310

Dashboard: show custom-domain email warning inline, not as a banner

6937192

Render the warning as help text under the email input (matching Calypso's altitude) instead of a Notice banner. Ref: DOTMSD-1310

Dashboard: render custom-domain warning as plain help text

50be0b7

Drop the custom has-warning styling; show the warning through the InputControl help prop with default styling. No bespoke CSS. Ref: DOTMSD-1310

Dashboard: warning as plain yellow help text, no icon

e956b82

Ref: DOTMSD-1310

Use the info icon and warning color.

3c841a3

StevenDufresne marked this pull request as ready for review July 2, 2026 05:45

StevenDufresne requested a review from a team as a code owner July 2, 2026 05:45

matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jul 2, 2026

claude Bot reviewed Jul 2, 2026

View reviewed changes

Comment thread client/dashboard/me/profile-personal-details/test/email-utils.test.ts Outdated

StevenDufresne added 2 commits July 2, 2026 15:03

Dashboard: drop isolated email-utils test, cover edge case in integra…

04a72d0

…tion test Per client/dashboard/docs/testing.md, simple utilities are covered via integration tests. Move the subdomain edge case into index.test.tsx. Ref: DOTMSD-1310

Dashboard: add subdomain edge case to email warning integration test

b2b123e

Ref: DOTMSD-1310

fushar reviewed Jul 2, 2026

View reviewed changes

Comment thread client/dashboard/me/profile-personal-details/email-section.tsx Outdated

Update client/dashboard/me/profile-personal-details/email-section.tsx

1792620

Co-authored-by: Ashar Fuadi <ashar.fuadi@automattic.com>

Dashboard: use single quotes for custom-domain warning string (lint)

ef01550

fushar approved these changes Jul 3, 2026

View reviewed changes

StevenDufresne added this pull request to the merge queue Jul 6, 2026

Merged via the queue into trunk with commit 746b23f Jul 6, 2026
15 checks passed

StevenDufresne deleted the add/dashboard-custom-domain-email-warning branch July 6, 2026 00:52

github-actions Bot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jul 6, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Dashboard: warn when Profile email uses a custom domain#112201

Dashboard: warn when Profile email uses a custom domain#112201
StevenDufresne merged 9 commits into
trunkfrom
add/dashboard-custom-domain-email-warning

StevenDufresne commented Jul 2, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Jul 2, 2026 •

edited by matticbot

Loading

Uh oh!

Uh oh!

fushar left a comment

Uh oh!

Uh oh!

StevenDufresne commented Jul 2, 2026

Uh oh!

github-actions Bot commented Jul 2, 2026

Uh oh!

fushar left a comment

Uh oh!

StevenDufresne commented Jul 3, 2026

Uh oh!

Uh oh!

a8ci18n commented Jul 6, 2026 •

edited

Loading

Uh oh!

a8ci18n commented Jul 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

StevenDufresne commented Jul 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed Changes

Why are these changes being made?

Considerations

Use a banner

Testing Instructions

Considerations

Pre-merge Checklist

Uh oh!

github-actions Bot commented Jul 2, 2026 • edited by matticbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

fushar left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

StevenDufresne commented Jul 2, 2026

Uh oh!

github-actions Bot commented Jul 2, 2026

Uh oh!

fushar left a comment

Choose a reason for hiding this comment

Uh oh!

StevenDufresne commented Jul 3, 2026

Uh oh!

Uh oh!

a8ci18n commented Jul 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

a8ci18n commented Jul 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

StevenDufresne commented Jul 2, 2026 •

edited

Loading

github-actions Bot commented Jul 2, 2026 •

edited by matticbot

Loading

a8ci18n commented Jul 6, 2026 •

edited

Loading