Skip to content

Hometask 5#72

Open
0ro wants to merge 8 commits intoromabelka:masterfrom
0ro:hometask_5
Open

Hometask 5#72
0ro wants to merge 8 commits intoromabelka:masterfrom
0ro:hometask_5

Conversation

@0ro
Copy link

@0ro 0ro commented Feb 4, 2018

No description provided.

const options = articles.map(article => ({
label: article.title,
value: article.id
const options = Object.keys(articles).map(i => ({
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше селектор, который достанет из в виде массива

...articles,
[articleId]: {
...article,
comments: [...comments, commentId]
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comments может быть undefined


const enhancer = composeEnhancers(
applyMiddleware(logger)
applyMiddleware(logger, idGenerator)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лучше логгер в конце ставить

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants