Skip to content
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

fix(useList): fix setLoading when request multiple times (#391) #397

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

JackWang032
Copy link
Collaborator

fix issue #391

@JackWang032 JackWang032 changed the title fix(useList): fix setLoading when request multiple times (#391) 【WIP】fix(useList): fix setLoading when request multiple times (#391) Oct 19, 2023
@mumiao
Copy link
Collaborator

mumiao commented Nov 6, 2023

这个状态咋是WIP,我看代码没问题了

@JackWang032
Copy link
Collaborator Author

这个状态咋是WIP,我看代码没问题了

跟修能review了下,想想有没有更优雅的写法,目前看解决竞态问题这样做是最好的了

@JackWang032 JackWang032 changed the title 【WIP】fix(useList): fix setLoading when request multiple times (#391) fix(useList): fix setLoading when request multiple times (#391) Nov 6, 2023
@mumiao
Copy link
Collaborator

mumiao commented Nov 7, 2023

+1

@mumiao mumiao merged commit bfc53ae into DTStack:master Nov 7, 2023
1 check passed
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.

None yet

2 participants