From 0432b03cbce46bf598aafb678edc5dab1fd3799e Mon Sep 17 00:00:00 2001 From: ilya-bkv Date: Thu, 8 Feb 2024 18:37:17 +0300 Subject: [PATCH] refactoring email check status --- public/locales/en/translation.json | 3 +- public/locales/ru/translation.json | 3 +- .../Author/AuthorBadge/AuthorBadge.tsx | 2 +- .../AuthModal/PasswordField/PasswordField.tsx | 3 +- src/components/Nav/AuthModal/RegisterForm.tsx | 139 ++++++++++-------- 5 files changed, 81 insertions(+), 69 deletions(-) diff --git a/public/locales/en/translation.json b/public/locales/en/translation.json index 85f2a6cd..d9f59e38 100644 --- a/public/locales/en/translation.json +++ b/public/locales/en/translation.json @@ -377,6 +377,8 @@ "There are unsaved changes in your publishing settings. Are you sure you want to leave the page without saving?": "There are unsaved changes in your publishing settings. Are you sure you want to leave the page without saving?", "This comment has not yet been rated": "This comment has not yet been rated", "This email is": "This email is", + "This email is not verified": "This email is not verified", + "This email is registered": "This email is registered", "This functionality is currently not available, we would like to work on this issue. Use the download link.": "This functionality is currently not available, we would like to work on this issue. Use the download link.", "This month": "This month", "This post has not been rated yet": "This post has not been rated yet", @@ -407,7 +409,6 @@ "Upload video": "Upload video", "Upload": "Upload", "Uploading image": "Uploading image", - "User with this email already exists": "User with this email already exists", "Username": "Username", "Userpic": "Userpic", "Users": "Users", diff --git a/public/locales/ru/translation.json b/public/locales/ru/translation.json index bc783798..04df15a2 100644 --- a/public/locales/ru/translation.json +++ b/public/locales/ru/translation.json @@ -399,6 +399,8 @@ "There are unsaved changes in your publishing settings. Are you sure you want to leave the page without saving?": "В настройках публикации есть несохраненные изменения. Уверены, что хотите покинуть страницу без сохранения?", "This comment has not yet been rated": "Этот комментарий еще пока никто не оценил", "This email is": "Этот email", + "This email is not verified": "Этот email не подтвержден", + "This email is registered": "Этот email уже зарегистрирован", "This functionality is currently not available, we would like to work on this issue. Use the download link.": "В данный момент этот функционал не доступен, бы работаем над этой проблемой. Воспользуйтесь загрузкой по ссылке.", "This month": "За месяц", "This post has not been rated yet": "Эту публикацию еще пока никто не оценил", @@ -429,7 +431,6 @@ "Upload video": "Загрузить видео", "Upload": "Загрузить", "Uploading image": "Загружаем изображение", - "User with this email already exists": "Пользователь с таким email уже существует", "Username": "Имя пользователя", "Userpic": "Аватар", "Users": "Пользователи", diff --git a/src/components/Author/AuthorBadge/AuthorBadge.tsx b/src/components/Author/AuthorBadge/AuthorBadge.tsx index 35537140..15255eb2 100644 --- a/src/components/Author/AuthorBadge/AuthorBadge.tsx +++ b/src/components/Author/AuthorBadge/AuthorBadge.tsx @@ -74,7 +74,7 @@ export const AuthorBadge = (props: Props) => { on( () => props.isFollowed, () => { - setIsFollowed(props.isFollowed.value) + setIsFollowed(props.isFollowed?.value) }, ), ) diff --git a/src/components/Nav/AuthModal/PasswordField/PasswordField.tsx b/src/components/Nav/AuthModal/PasswordField/PasswordField.tsx index 331c4009..c2e0e3bc 100644 --- a/src/components/Nav/AuthModal/PasswordField/PasswordField.tsx +++ b/src/components/Nav/AuthModal/PasswordField/PasswordField.tsx @@ -13,6 +13,7 @@ type Props = { errorMessage?: (error: string) => void onInput: (value: string) => void variant?: 'login' | 'registration' + disableAutocomplete?: boolean } export const PasswordField = (props: Props) => { @@ -69,7 +70,7 @@ export const PasswordField = (props: Props) => { id="password" name="password" disabled={props.disabled} - autocomplete="current-password" + autocomplete={props.disableAutocomplete ? 'one-time-code' : 'current-password'} type={showPassword() ? 'text' : 'password'} placeholder={props.placeholder || t('Password')} onInput={(event) => handleInputChange(event.currentTarget.value)} diff --git a/src/components/Nav/AuthModal/RegisterForm.tsx b/src/components/Nav/AuthModal/RegisterForm.tsx index 6b0b16f8..c17901c2 100644 --- a/src/components/Nav/AuthModal/RegisterForm.tsx +++ b/src/components/Nav/AuthModal/RegisterForm.tsx @@ -1,8 +1,8 @@ import type { JSX } from 'solid-js' +import { Show, createEffect, createMemo, createSignal } from 'solid-js' import type { AuthModalSearchParams } from './types' import { clsx } from 'clsx' -import { Show, createEffect, createMemo, createSignal } from 'solid-js' import { useLocalize } from '../../../context/localize' import { useSession } from '../../../context/session' @@ -17,6 +17,8 @@ import { email, setEmail } from './sharedLogic' import styles from './AuthModal.module.scss' +type EmailStatus = 'not verified' | 'verified' | 'registered' | '' + type FormFields = { fullName: string email: string @@ -44,17 +46,6 @@ export const RegisterForm = () => { const authFormRef: { current: HTMLFormElement } = { current: null } - const checkEmail = async (address: string) => { - const result: string = await isRegistered(address) - if (result) setEmailStatus((_s) => result) - } - - const handleEmailBlur = () => { - if (validateEmail(email())) { - checkEmail(email()) - } - } - const handleNameInput = (newName: string) => { setFullName(newName) } @@ -100,7 +91,6 @@ export const RegisterForm = () => { return } - setIsSubmitting(true) try { const opts = { @@ -111,26 +101,7 @@ export const RegisterForm = () => { redirect_uri: window.location.origin, } const { errors } = await signUp(opts) - if (errors?.some((error) => error.message.includes('has already signed up'))) { - setValidationErrors((prev) => ({ - ...prev, - email: ( - <> - {t('User with this email already exists')},{' '} - - changeSearchParams({ - mode: 'login', - }) - } - > - {t('sign in')} - - - ), - })) - } + if (errors) return setIsSuccess(true) } catch (error) { console.error(error) @@ -142,6 +113,72 @@ export const RegisterForm = () => { console.debug(emailStatus()) }) + const handleCheckEmailStatus = (status: EmailStatus | string) => { + switch (status) { + case 'not verified': + setValidationErrors((prev) => ({ + ...prev, + email: ( + <> + {t('This email is not verified')},{' '} + resendVerifyEmail({ email: email(), identifier: 'basic_signup' })} + > + {t('resend confirmation link')} + + + ), + })) + break + + case 'verified': + setValidationErrors((prev) => ({ + ...prev, + email: ( + <> + {t('This email is verified')},{' '} + changeSearchParams({ mode: 'login' })}> + {t('enter')} + + + ), + })) + break + case 'registered': + setValidationErrors((prev) => ({ + ...prev, + email: ( + <> + {t('This email is registered')},{'. '} + changeSearchParams({ mode: 'send-reset-link' })}> + {t('Set the new password').toLocaleLowerCase()} + + + ), + })) + break + default: + setValidationErrors((prev) => ({ + ...prev, + email: ( + <> + {t('This email is')} {status.length ? status : 'странный ¯\\_(ツ)_/¯'} + + ), + })) + break + } + } + + const handleEmailBlur = async () => { + if (validateEmail(email())) { + const checkResult = await isRegistered(email()) + console.log('!!! checkResult:', checkResult) + handleCheckEmailStatus(checkResult) + } + } + return ( <> @@ -162,11 +199,11 @@ export const RegisterForm = () => { name="fullName" type="text" placeholder={t('Full name')} - autocomplete="" + autocomplete="one-time-code" onInput={(event) => handleNameInput(event.currentTarget.value)} /> - +
{validationErrors().fullName}
 @@ -179,46 +216,18 @@ export const RegisterForm = () => { handleEmailInput(event.currentTarget.value)} onBlur={handleEmailBlur} />
{validationErrors().email}
- - {t('This email is')} {emailStatus() ? t(emailStatus()) : ''},{' '} - - - changeSearchParams({ mode: 'login' })}> - {t('enter')} - - - - resendVerifyEmail({ email: email(), identifier: 'basic_signup' })} - > - {t('resend confirmation link')} - - - - - changeSearchParams({ - mode: 'send-reset-link', - }) - } - > - {t('Set the new password').toLocaleLowerCase()} - - setPasswordError(err)} onInput={(value) => setPassword(value)}