Skip to content

Fix clear cache #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Aug 28, 2017
Merged

Fix clear cache #70

merged 4 commits into from
Aug 28, 2017

Conversation

paranoidjk
Copy link
Member

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 79.2% when pulling 1c8d7cb on fix-clear-cache into 790bfb8 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 79.6% when pulling e3b3757 on fix-clear-cache into 790bfb8 on master.

@@ -376,6 +376,7 @@ class Select extends Component {
}
event.stopPropagation();
this._cacheTreeNodesStates = 'no';
this._checkedNodes = [];
Copy link
Member Author

Choose a reason for hiding this comment

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

@afc163 源码改动就这一行。

@afc163 afc163 merged commit f2fbcb2 into master Aug 28, 2017
@paranoidjk
Copy link
Member Author

@afc163 建议 setting 里面把 merge 去掉,只允许 sqush and merge,我没权限改.

@paranoidjk paranoidjk deleted the fix-clear-cache branch August 28, 2017 09:05
@afc163
Copy link
Member

afc163 commented Aug 28, 2017

我也没权限

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.

Cleared data show in TreeSelect again when use AllowClear button
3 participants