Skip to content

Comments

Code Review#2

Open
inomdzhon wants to merge 1 commit intoilovesg:mainfrom
inomdzhon:comments
Open

Code Review#2
inomdzhon wants to merge 1 commit intoilovesg:mainfrom
inomdzhon:comments

Conversation

@inomdzhon
Copy link

@inomdzhon inomdzhon commented Oct 23, 2022

Что было сделано?

  • Вынес localStorage в абстракцию LocalStorage – теперь один ключ это одна сущность. Это уменьшает коллизию ключей в localStorage.
  • Верификация телефона теперь привязана на field.type === "tel", а не на последний шаг.
  • Вынес данные на уровень приложения в папку /mock

Что можно улучшить?

  • react-final-form умеет в хуки, поэтому можно было бы на них переписать вместо render props
  • создание formStore можно вынести в отдельный файлик на уровне features/registration-form/ – подумать/изучить как такое организовать (ключевые слова cache, store)
  • верификацию телефона можно попробовать сделать полностью обособленным – сейчас при проверке рендерится два компонента, а можно было бы их объединить (типа это тот же RegistrationFormItem, но расширенным функционалом)
    item.type === "tel" ? (
    <React.Fragment key={item.field}>
    <RegistrationFormItem item={item} />
    <RegistrationFormPhoneVerification
    code={phoneCode}
    setCode={onPhoneCodeChange}
    />
    </React.Fragment>
    ) : (

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.

1 participant