Ht 7 #58
Ht 7 #58
Conversation
# Conflicts: # README.md
| @@ -6,7 +6,24 @@ const orderSelector = (state) => state.order; | |||
| const productsSelector = (state) => state.products.entities; | |||
| const reviewsSelector = (state) => state.reviews.entities; | |||
| const usersSelector = (state) => state.users.entities; | |||
| const routerSelector = (state) => state.router; | |||
There was a problem hiding this comment.
в connected-react-router есть свои селекторы, которые можно так же использовать
| loaded: false, | ||
| }; | ||
|
|
||
| export default (state = initStatus, action) => { |
There was a problem hiding this comment.
это все можно было поместить в редьюсер для ордера, который у нас уже есть
| const { type, payload } = action; | ||
| switch (type) { | ||
| case CHANGE_CURRENCY: | ||
| if (currencies.some((cur) => cur.id === payload)) |
There was a problem hiding this comment.
если currencies изначально сделать объектом, а не массивом - будет намного проще
| @@ -18,7 +22,9 @@ const Product = ({ product, amount, increment, decrement }) => { | |||
| <div> | |||
| <h4 className={styles.title}>{product.name}</h4> | |||
| <p className={styles.description}>{product.ingredients.join(', ')}</p> | |||
| <div className={styles.price}>{product.price} $</div> | |||
| <div className={styles.price}> | |||
| {product.price} {sign} | |||
There was a problem hiding this comment.
тут цена в долларах, на сколько я понимаю и к ней просто дописывается знак текущей валюты
|
|
||
| const initStatus = { ...arrToMap(currencies) }['dollar']; | ||
|
|
||
| export default (state = initStatus, action) => { |
There was a problem hiding this comment.
это необходимо было сделать на контексте
| @@ -11,3 +12,4 @@ export const LOAD_USERS = 'LOAD_USERS'; | |||
| export const REQUEST = '_REQUEST'; | |||
| export const SUCCESS = '_SUCCESS'; | |||
| export const FAILURE = '_FAILURE'; | |||
| export const CHANGE_CURRENCY = 'CHANGE_CURRENCY'; | |||
There was a problem hiding this comment.
я не нашел где используется это екшен
There was a problem hiding this comment.
Да времени особенно не было селектор валют прикрутить, и некоторые другие детали,.т.к. работу никто не отменял. Сделал что успел. Можно просто store.dispatch(), так и проверял
| <Button | ||
| onClick={() => | ||
| createOrder( | ||
| orderProducts.map(({ product, amount }) => ({ |
There was a problem hiding this comment.
Это все должно быть в екшене. Сейчас компонент знает в каком виде нам нужно отправлять данные на апи
| }); | ||
|
|
||
| export const createOrder = (items) => async (dispatch, getState) => { | ||
| const state = getState(); |
2+3 пункты, кроме страницы success