Conversation
| return comments | ||
| .set('loading', false) | ||
| .set('loaded', true) | ||
| .set('entities', arrToMap(response, CommentRecord)); |
There was a problem hiding this comment.
Я думаю, что здесь я возвращаю неправильный объект, т.к. у меня все комментарии оказываются в entities.
There was a problem hiding this comment.
сейчас загружая комменты для одной статьи ты перезатираешь прошлые; используй .mergeIn
| const body = comments.entities.size > 0 ? ( | ||
| <ul> | ||
| {comments.map(id => <li key = {id}><Comment id = {id} /></li>)} | ||
| {comments.entities.valueSeq().map(comment => <li key = {comment.id}><Comment comment={comment} /></li>)} |
There was a problem hiding this comment.
сделай для этого селектор, старайся не завязываться в компонентах на api immutable
| return comments | ||
| .set('loading', false) | ||
| .set('loaded', true) | ||
| .set('entities', arrToMap(response, CommentRecord)); |
There was a problem hiding this comment.
сейчас загружая комменты для одной статьи ты перезатираешь прошлые; используй .mergeIn
|
|
||
| const ReducerRecord = Record({ | ||
| entities: arrToMap([], CommentRecord), | ||
| loading: false, |
There was a problem hiding this comment.
здесь не достаточно повесить loading на весь comments, ведь ты для конкрентной статьи загружаешь
| case LOAD_ALL_COMMENTS + SUCCESS: | ||
| return comments | ||
| .set('loading', false) | ||
| .set('loaded', true) |
There was a problem hiding this comment.
Почему loaded === true, ведь ты загрузил только для одной статьи?
2 participants
Поломал добавление комментариев.