fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.

https://demo.fullstack-development.com/

MIT License

91 stars 13 forks source link

63 react 18next configure #71

Closed clicktronix closed 5 years ago

clicktronix commented 5 years ago

63

in19farkt commented 5 years ago

@clicktronix а что с ПРом? Он у тебя в статусе WIP.

clicktronix commented 5 years ago

@in19farkt забыл про него уже, мы подрубаем переводы из locize.io? Там вроде какое то количество ключей бесплатно, плюс надо подумать как типизировать это дело. Можно оставить англ переводы в проекте, на фоллбэк и как раз для типов

clicktronix commented 5 years ago

Переписал ПР для мержа в мастер. Тесты пока поломаны из-за добавления пропсов перевода, не стал здесь исправлять, чтоб ПР не захламлять, поправлю тесты в отдельной ветке, заодно напилю новых для i18n. @in19farkt @chmnkh @NikitaRzm @Znack посмотри по возможности пр плз

Znack commented 5 years ago

а как без тестов работающих мержить в мастер? :) Это странные приоритеты, не добавлять тесты, чтобы не захламлять, лучше тогда разбить функциональность на два ПР, но каждый с норм тестами :)

clicktronix commented 5 years ago

Тесты поправил, новые на локализацю в другом пр сделаю

Znack commented 5 years ago

@chmnkh не смотрел ПР ещё?

clicktronix commented 5 years ago

@in19farkt мержим его?)

in19farkt commented 5 years ago

@clicktronix конфликты :(

clicktronix commented 5 years ago

@in19farkt порешал

in19farkt commented 5 years ago

@chmnkh @clicktronix надо локально всё проверить, и если всё нормально работает, тесты норм и ничего не поломали, то наверное можно мержнуть.

Znack commented 5 years ago

@chmnkh @clicktronix а какой тут статус сейчас? :)

clicktronix commented 5 years ago

Я тесты поправил, все протыкал, вроде работает. Но у нас изоморфность в мастере поломана, стили не рендерятся

chmnkh commented 5 years ago

запамятовал про пр, завтра гляну тоже

chmnkh commented 5 years ago

приложение и тесты проверил: все нормас