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

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

Где authLegalNote?

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

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

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

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

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

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

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

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

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

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

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

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

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

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

4040
52 комментария

Бан в подайте "Сломалось" на три дня, по причине: Я художник, я так вижу.

34
Ответить

Ну вот да, разве что по этому.

7
Ответить

Предлагаю краудсорсить кодревью, будем всем дтф смотреть ваши пулреквесты х)

12
Ответить

На самом деле изи, важно лишь желание. Я вот каждый день сверяюсь по фронту у них.
upd: Зачем я это делаю? Смотри в блоге, йопта.

1
Ответить

@Филипп Концаренко @Сломалось Серьезно блядь, вы сделали хуже. Зачем?

3
Ответить

@Vladislav Tsyplukhin наймите кого-нибудь на код ревью, правда.

3
Ответить

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

2
Ответить