mirror of
https://github.com/toeverything/AFFiNE.git
synced 2026-03-22 23:30:36 +08:00
fix: Refine verify email dialog for verify and change email flows (#14671)
### Summary This PR improves the verify email dialog by giving the verify-email and change-email flows distinct messaging instead of reusing the same generic copy. ### What changed * Use flow-specific body copy in the verify email dialog * Keep the existing action-specific subtitle behavior for: * Verify email * Change email * Update the English i18n strings so each flow explains the correct intent: * Verify email focuses on confirming email ownership * Change email focuses on securely starting the email-change process ### Why The previous dialog message was shared across both flows, which made the change-email experience feel ambiguous. This update makes the intent clearer for users and better matches the action they are taking. https://www.loom.com/share/c64c20570a8242358bd178a2ac50e413 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Improved clarity in email verification and email change dialog messages to better explain the confirmation process and link purpose. * Enhanced distinction between email verification and email change workflows with context-specific messaging. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
@@ -108,10 +108,18 @@ export const VerifyEmailDialog = ({
|
||||
>
|
||||
<AuthHeader
|
||||
title={serverName}
|
||||
subTitle={t['com.affine.settings.email.action.change']()}
|
||||
subTitle={
|
||||
changeEmail
|
||||
? t['com.affine.settings.email.action.change']()
|
||||
: t['com.affine.settings.email.action.verify']()
|
||||
}
|
||||
/>
|
||||
<AuthContent>
|
||||
<p>{t['com.affine.auth.verify.email.message']({ email })}</p>
|
||||
<p>
|
||||
{changeEmail
|
||||
? t['com.affine.auth.change.email.message']({ email })
|
||||
: t['com.affine.auth.verify.email.message']({ email })}
|
||||
</p>
|
||||
<AuthInput
|
||||
label={t['com.affine.settings.email']()}
|
||||
disabled={true}
|
||||
|
||||
@@ -262,7 +262,7 @@
|
||||
"com.affine.appearanceSettings.translucentUI.title": "Translucent UI on the sidebar",
|
||||
"com.affine.appearanceSettings.showLinkedDocInSidebar.title": "Show linked doc in sidebar",
|
||||
"com.affine.appearanceSettings.showLinkedDocInSidebar.description": "Control whether to show the structure of linked docs in the sidebar.",
|
||||
"com.affine.auth.change.email.message": "Your current email is {{email}}. We'll send a temporary verification link to this email.",
|
||||
"com.affine.auth.change.email.message": "Your current email is {{email}}. We'll send a confirmation link there first so you can securely switch to a new email address.",
|
||||
"com.affine.auth.change.email.page.subtitle": "Please enter your new email address below. We will send a verification link to this email address to complete the process.",
|
||||
"com.affine.auth.change.email.page.success.subtitle": "Congratulations! You have successfully updated the email address associated with your AFFiNE Cloud account.",
|
||||
"com.affine.auth.change.email.page.success.title": "Email address updated!",
|
||||
@@ -347,7 +347,7 @@
|
||||
"com.affine.auth.toast.message.signed-in": "You have been signed in, start to sync your data with AFFiNE Cloud!",
|
||||
"com.affine.auth.toast.title.failed": "Unable to sign in",
|
||||
"com.affine.auth.toast.title.signed-in": "Signed in",
|
||||
"com.affine.auth.verify.email.message": "Your current email is {{email}}. We'll send a temporary verification link to this email.",
|
||||
"com.affine.auth.verify.email.message": "Your current email is {{email}}. We'll send a verification link to this email so you can confirm it belongs to you.",
|
||||
"com.affine.backButton": "Back",
|
||||
"com.affine.banner.content": "This demo is limited. <1>Download the AFFiNE Client</1> for the latest features and Performance.",
|
||||
"com.affine.banner.local-warning": "Your local data is stored in the browser and may be lost. Don't risk it - enable cloud now!",
|
||||
|
||||
Reference in New Issue
Block a user