IgorStolovoi / React

0 stars 0 forks source link

RHW3 #3

Open nelilika opened 3 years ago

nelilika commented 3 years ago

по функциональности:

код ревью

мне нравится что вынес валидацию в отдельный файл - это хорошая практика. чего для паролей валидация в прямо в jsx делается? _ у тебя бета версия Material UI стоит - это не гуд, выбери что-то одно либо 4 либо 5 и неправильные импорты

import List from "../../node_modules/@material-ui/core/List";
import ListItem from "../../node_modules/@material-ui/core/ListItem";
import ListItemText from "../../node_modules/@material-ui/core/ListItemText";

никаких относительных импортов для пакетов - только по названию @material-ui/core/List

IgorStolovoi commented 3 years ago

-что-то ковырял в 3-м шагу и забыл поменять на 1) -валидацию пароля не вынес, т.к не понял как сравнить два пароля в yup. А функционал валидации, предоставляемые hook-foorm, не работает если есть resolver)

nelilika commented 2 years ago

@JuicyBlunt по поводу валидации зависимых полей посмотри в сторону метода ".when()"