ExecutionLLC / gen-bs

gen-bs
1 stars 1 forks source link

Убрать варнинги при запуске приложения #897

Closed ZzDmitry closed 7 years ago

ZzDmitry commented 7 years ago

В консоль сыпется много варнингов, что может помешать увидеть ошибки. Часть должна уйти после обновления реакта (#896), часть с ним не связана.

ZzDmitry commented 7 years ago

перечисленное даёт нам 21 варнинг на старте

ZzDmitry commented 7 years ago

п.9. Нельзя просто взять и вернуть в рендерере массив (как в renderTempRow), у каждого элемента должен быть key.

ZzDmitry commented 7 years ago

П.6. и п.7. сделаны в #896

ZzDmitry commented 7 years ago

П.1. и 2. исправлены одной правкой. Почему-то вместо вызова экшена storeCurrentLanguageId, который только сохраняет текущий язык в стейте приложения (после получения оного для пользователя или при начальной инициализации приложения) вызывался акшен setCurrentLanguageId, который задаёт язык пользователя. На самом деле там ещё нужет вызов setLanguage из redux-polyglot. Всё это сделано, варнингов нет.

ZzDmitry commented 7 years ago

Ждём мержа пулреквеста #914 и закрытия #896, мержим мастер сюда и проверяем отсутствие варнингов.