"Код-ревью" новой формы авторизации
Мало того, что вы ее усложнили, так еще и оставили уйму ошибок.
Где authLegalNote?
Главный экран social-registration сыплет ошибками на handlers.onSuccess. Ибо логично, ибо нет ручек там. Думаете это выражение спасает от этого? В форме уж тогда сделайте проверку на наличие функции.
Почему auth-form у вас отображается, если нет shouldDisplayMessage? Скорее всего условие не так построено, верно?
В новых text-input вы потеряли атрибут name. Это мешает автозаполнению форм.
Разный способ прокидывания экранов в компонентах. В одном месте мы делаем так:
В другом мы делаем так:
Верстка ссылок на другие экраны отличается.
В одном месте у нас <a>, в другом <button>.
У вас вообще нет код ревью?
41 комментарий