63 react 18next configure #71
Conversation
|
@clicktronix а что с ПРом? Он у тебя в статусе WIP. |
|
@in19farkt забыл про него уже, мы подрубаем переводы из locize.io? Там вроде какое то количество ключей бесплатно, плюс надо подумать как типизировать это дело. Можно оставить англ переводы в проекте, на фоллбэк и как раз для типов |
|
Переписал ПР для мержа в мастер. Тесты пока поломаны из-за добавления пропсов перевода, не стал здесь исправлять, чтоб ПР не захламлять, поправлю тесты в отдельной ветке, заодно напилю новых для i18n. @in19farkt @chmnkh @NikitaRzm @Znack посмотри по возможности пр плз |
|
а как без тестов работающих мержить в мастер? :) Это странные приоритеты, не добавлять тесты, чтобы не захламлять, лучше тогда разбить функциональность на два ПР, но каждый с норм тестами :) |
|
Тесты поправил, новые на локализацю в другом пр сделаю |
|
@chmnkh не смотрел ПР ещё? |
src/features/profile/view/containers/ProfileEdit/ProfileEdit.tsx
Outdated
Show resolved
Hide resolved
...eatures/repositoriesSearch/view/containers/RepositoriesSearchForm/RepositoriesSearchForm.tsx
Outdated
Show resolved
Hide resolved
src/shared/view/components/SearchForm/SearchSettingsDialog/SearchSettingsDialog.tsx
Outdated
Show resolved
Hide resolved
|
@in19farkt мержим его?) |
|
@clicktronix конфликты :( |
|
@in19farkt порешал |
|
@chmnkh @clicktronix надо локально всё проверить, и если всё нормально работает, тесты норм и ничего не поломали, то наверное можно мержнуть. |
|
@chmnkh @clicktronix а какой тут статус сейчас? :) |
|
Я тесты поправил, все протыкал, вроде работает. Но у нас изоморфность в мастере поломана, стили не рендерятся |
|
запамятовал про пр, завтра гляну тоже |
src/features/repositoriesSearch/view/components/RepositoryPreview/RepositoryPreview.tsx
Outdated
Show resolved
Hide resolved
|
приложение и тесты проверил: все нормас |
#63