Skip to content

Hometask5#74

Open
Roughtron wants to merge 3 commits intoromabelka:masterfrom
Roughtron:hometask5
Open

Hometask5#74
Roughtron wants to merge 3 commits intoromabelka:masterfrom
Roughtron:hometask5

Conversation

@Roughtron
Copy link

No description provided.

toggleOpen={this.toggleOpenItemMemoized(article.id)}
/>
</li>)
if (!Object.keys(articles).length) return <h3>No Articles</h3>
Copy link
Owner

Choose a reason for hiding this comment

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

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

const { type, payload } = action

switch (type) {
case ADD_COMMENT: {
Copy link
Owner

Choose a reason for hiding this comment

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

через мидлвары будет проходить каждый экшин, они должны быть максимально общими, завязывать на конкретные экшины - не лучшее решение


const enhancer = composeEnhancers(
applyMiddleware(logger)
applyMiddleware(logger, uniqueIds)
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