Redux naming convention (была постановка, было решение, но не то) кто как делает + немного про жиЗь?

Question

[Evgenyx82]

Привет, товарищи.

Был такой сабж

Приложение должно состоять из двух страниц:

1. Первая страница отображает список всех пользователей бота. Для каждого пользователя отображается аватар и имя.
   
2. Вторая страница отображает детальную информацию о пользователе и позволяет администратору отредактировать имя пользователя.
   

Можно использовать любой набор инструментов, фреймворков, библиотек и даже языков. Детали, которых не хватает в задаче, можно выбрать на свое усмотрение, включая дизайн.

В результате получилось

вот это https://github.com/Evgenyx82/Chatfuel

Ответ был такой

Спасибо за интерес к нашей вакансии и за время, инвестированное в выполнение тестового задания! Мы внимательно его изучили и решили, что не готовы двигаться с вами дальше по данной вакансии.

Мы оцениваем не только работоспособность решения, но и стиль программирования: простоту и понятность кода, архитектурную декомпозицию, аккуратность, а также использование возможностей выбранного языка. Ваш код, к сожалению, не соответствует нашим ожиданиям по этой части.

На что конкретно мы обратили внимание:

1. вы выбрали архитектуру flux и используете redux но сетевые запросы продолжаете делать в компонентах. метод onSaveButton = () => {
   
2. вы колокейтите названия actions и actions creators что в большом приложение вызовет конфликт названий для actions
   

Что я думаю по этому поводу

1. Ладно, запарился был уже вечер, поправил :)
   
2. Точно не понял, что он тут пишет, но в целом это видимо про naming convention, как поправить так, чтобы вопрос номер два не возникал? HOME_USERS_FETCH, USER_USERS_FETCH?
   
3. Так же, хотелось бы узнать общие впечатления от такого респонса, исходя из постановки и решения? )
   

Comments

[davidnum95]

Зачем вы в редьюсерах используете Object.assign()? Используйте spread оператор из es6.

[Evgenyx82]

davidnum95, Вопрос, не про это. )