Lesson6 #84
Lesson6 #84
Conversation
| import produce from 'immer'; | ||
| import { CHANGE_RESTAURANT_TAB } from '../constants'; | ||
|
|
||
| export default produce((draft = { restaurantActiveTab: 'menu' }, action) => { |
There was a problem hiding this comment.
у нас эта вкладка есть в роутинге, не совсем понимаю зачем ее сохранять в редакс
There was a problem hiding this comment.
Хотелось сохранить логику, чтобы при переключении ресторана не менялась выбраная подвкладка и при этом не хотелось показывать эту подвкладку родительским компонентам.
Как при переключении по Navlink получить предыдущую вкладку не придумалось, пришлось сохранять в редакс.
| @@ -22,7 +22,8 @@ export default (state = initialState, action) => | |||
| draft.loading[restId] = false; | |||
| draft.loaded[restId] = true; | |||
| draft.error = null; | |||
| Object.assign(draft.entities, arrToMap(data)); | |||
| const dataWithRestId = data.map((obj) => ({ ...obj, restId })); | |||
| Object.assign(draft.entities, arrToMap(dataWithRestId)); | |||
There was a problem hiding this comment.
Из данных, которые у нас лежат в редьюсере ресторанов мы можем однозначно сказать какой продукт из какого ресторана. Теперь эта информация у нас есть в двух местах, тут и в ресторанах. Так делать не стоит.
There was a problem hiding this comment.
Вот про это очень интересно, мне казалось, что обратные ссылки это нормально :))
| }; | ||
| }; | ||
|
|
||
| export const getActiveRestaurantTab = () => (_dispatch, getState) => { |
There was a problem hiding this comment.
Тут собственно так сделано, чтобы получить значение однократно при перерендере компоненты с новым рестораном, а не при каждой смене таба. Не очень понимаю, как можно для этого использовать селектор, можно подробнее?
| path="/restaurants/:restId/reviews" | ||
| component={() => <Reviews reviews={reviews} restId={id} />} | ||
| /> | ||
| <Redirect to={`/restaurants/:restId/${getActiveRestaurantTab()}`} />; |
There was a problem hiding this comment.
тут нужно просто редиректить на меню
There was a problem hiding this comment.
Вот собственно ради этого все и затевалось :)
если по умолчанию редиректить на меню, у нас относительно текущей версии приложения потеряется сохранение выбранной подвкладки ресторана. А менять эту логику не хотелось.
No description provided.