HT-5 kostenevich #71
Open
HT-5 kostenevich #71
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Смог пофиксить, проблема была как раз таки в том что не подтягивал ревью на уровне ресторана + проверка на рендер "NO DATA :(" была в && а не || и оно скипало из-за этого useEffect т.к. тот просто не успевал подтянуть нужные данные. |
koretskiyav
reviewed
Oct 29, 2021
| @@ -13,11 +20,19 @@ class Menu extends Component { | |||
|
|
|||
| state = { error: null }; | |||
|
|
|||
| componentDidMount() { | |||
There was a problem hiding this comment.
еще нужно обработать смену ресторана в componentDidUpdate, если убрать передачу key в Menu, то без componentDidUpdate не будут грузиться новые продукты
koretskiyav
reviewed
Oct 29, 2021
| }; | ||
|
|
||
| export default connect(null, mapDispatchToProps)(Reviews); | ||
| const mapDispatchToProps = (dispatch, props) => ({ | ||
| loadReviews: () => dispatch(loadReviews(props.restId)), |
There was a problem hiding this comment.
в компоненте идет вызов loadReviews(restId), тут можно простую форму mapDispatchToProps использовать с объектом
koretskiyav
reviewed
Oct 29, 2021
| const mapStateToProps = (state, props) => ({ | ||
| loading: productLoadingSelector(state, props), | ||
| loaded: productLoadedSelector(state, props), | ||
| restId: props.restId, |
There was a problem hiding this comment.
в компонент и так прийдет пропа restId
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
У меня есть баг, при котором только первый компонент REVIEWS проходит через useEffect, где подтягивает ревьюхи с апи. Если же перейти на табу ревью другого ресторана то при отрисовке нового компонента REVIEWS он не проходит через useEffect и получается что данные не подтягиваются, но последующие функции вызываются и идёт краш из-за того что нет подтянутых данных. Краш можно пофиксить тем, что подтягивать ревью ещё на Ресторане (оно в любом случае так правильно, что бы рендерился баннер адекватно), но по идее баг останется с useEffect'ом. Если не пофикшу до занятий, объясните плс в чём моя ошибка, ибо уже часа 2.5 гуглю и не могу найти решение, мозг совсем поплавился(