"Код-ревью" новой формы авторизации

Мало того, что вы ее усложнили, так еще и оставили уйму ошибок.

Где authLegalNote?

Главный экран social-registration сыплет ошибками на handlers.onSuccess. Ибо логично, ибо нет ручек там. Думаете это выражение спасает от этого? В форме уж тогда сделайте проверку на наличие функции.

"Код-ревью" новой формы авторизации

Почему auth-form у вас отображается, если нет shouldDisplayMessage? Скорее всего условие не так построено, верно?

"Код-ревью" новой формы авторизации

В новых text-input вы потеряли атрибут name. Это мешает автозаполнению форм.

"Код-ревью" новой формы авторизации

Разный способ прокидывания экранов в компонентах. В одном месте мы делаем так:

"Код-ревью" новой формы авторизации

В другом мы делаем так:

"Код-ревью" новой формы авторизации

Верстка ссылок на другие экраны отличается.

"Код-ревью" новой формы авторизации
"Код-ревью" новой формы авторизации

В одном месте у нас <a>, в другом <button>.

У вас вообще нет код ревью?

40
41 комментарий