Feature/ht5 #75
Feature/ht5 #75
Conversation
| @@ -17,20 +24,32 @@ class Menu extends Component { | |||
| this.setState({ error }); | |||
| } | |||
|
|
|||
| componentDidMount({ prevId } = {}) { | |||
There was a problem hiding this comment.
в componentDidMount ничего не приходит, это у нас самый первый рендер компонента
There was a problem hiding this comment.
@koretskiyav - так и есть. но за счет того, что мы перемонтируем меню каждый раз (key={id}), didMount отрабатывает каждое переключение ресторана и инициирует подгрузку продуктов.
There was a problem hiding this comment.
в данном случае, наверное, можно было не делать проверок в принципе, просто дергать за загрузку.
| } from './constants'; | ||
|
|
||
| const loadedItems = {}; |
There was a problem hiding this comment.
лучше данные. о том, что у нас уже загружено хранить в редаксе
| return { | ||
| ...state, | ||
| activeId: data[0].id, | ||
| entities: arrToMap(data), |
There was a problem hiding this comment.
не совсем понимаю, как это работает, если мы ревьюхи грузить только по одному разу за счет loadedItems, но при этом мы записываем только данный из последней загрузки
There was a problem hiding this comment.
@koretskiyav - это никак не работает, это поле не нужно вообще. результат копипаста из редьюсера ресторанов и спешки.
можно было много чего оптимизировать - редьюсеры, экшны, компоненты, но не было времени, сделал только основное