Skip to content

Conversation

@weifu2
Copy link
Contributor

@weifu2 weifu2 commented Jan 22, 2019

No description provided.

this.setState(previousState => {
return {
notices: previousState.notices.filter(notice => notice.key !== key),
notices: previousState.notices.filter(notice => notice.key !== key && notice.updateKey !== key),
Copy link
Member

Choose a reason for hiding this comment

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

You are right!
But maybe we should change
const close = createChainedFunction(remove.bind(this, key), onClose);
to
const close = createChainedFunction(remove.bind(this, notice.key), onClose);
is more better solution.

@codecov
Copy link

codecov bot commented Jan 23, 2019

Codecov Report

Merging #428 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #428 +/- ## ======================================= Coverage 87.62% 87.62% ======================================= Files 145 145 Lines 4744 4744 Branches 1404 1404 ======================================= Hits 4157 4157 Misses 543 543 Partials 44 44

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1813996...bda0871. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Jan 23, 2019

Codecov Report

Merging #428 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #428 +/- ## ======================================= Coverage 87.62% 87.62% ======================================= Files 145 145 Lines 4744 4744 Branches 1404 1404 ======================================= Hits 4157 4157 Misses 543 543 Partials 44 44

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1813996...bda0871. Read the comment docs.

@tangjinzhou tangjinzhou merged commit 159c549 into vueComponent:master Jan 23, 2019
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants