Hm4 kazmin #40
Hm4 kazmin #40
Conversation
| return TYPE_ARTICLE | ||
| } | ||
|
|
||
| constructor(id, type = CommentHolder.TYPE_ARTICLE) { |
There was a problem hiding this comment.
Как по мне - перемудрил с этим. Тем более ты пользуешься тем, что у нас key={id}, но ты не можешь быть в этом уверен в самом компоненте.
There was a problem hiding this comment.
тогда можно просто передавать entityId + entityOwner (entityHolder, etc.)
потому что если мы не будем делать абстракцию и потом компонент коментариев будет использоватся например списком новостей, прийдется компонент коментариев рефакторить
| export default (store) => (next) => (action) => { | ||
| const { type, payload } = action | ||
|
|
||
| if (type === ADD_COMMENT) { |
There was a problem hiding this comment.
через мидлвары будет проходить каждый экшин, они должны быть максимально общими, завязывать на конкретные экшины - не лучшее решение
There was a problem hiding this comment.
тогда просто подмешивать в каждый екшене в payload generatedId ?
| const articleId = payload.holder.getId() | ||
| const article = articleState[articleId] | ||
|
|
||
| article.comments = [].concat(article.comments) |
There was a problem hiding this comment.
article тоже по ссылке не меняй
There was a problem hiding this comment.
лучшее решение думаю deepClone (lodash), и в коменты подмешивать новый commentId
ну и если говорить о разбиении, comments - не лучшний нейминг, скорее commentIds
No description provided.