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

feat: support AbortController in waitForSelector #10018

Merged
merged 1 commit into from Apr 18, 2023
Merged

Conversation

OrKoN
Copy link
Collaborator

@OrKoN OrKoN commented Apr 13, 2023

Closes #9709

@OrKoN OrKoN force-pushed the orkon/abort-controller branch 2 times, most recently from 530b831 to 07ae04e Compare April 13, 2023 08:20
@OrKoN
Copy link
Collaborator Author

OrKoN commented Apr 13, 2023

Blocked on deprecation of Node 14 in #10019

I guess we can ship it since it is an optional feature and AbortController can be enabled on Node 14.

@OrKoN OrKoN enabled auto-merge (squash) April 18, 2023 13:57
@OrKoN OrKoN merged commit 9109b76 into main Apr 18, 2023
30 checks passed
@OrKoN OrKoN deleted the orkon/abort-controller branch April 18, 2023 16:45
This was referenced Apr 18, 2023
@MoLow
Copy link

MoLow commented May 1, 2023

Hey, thanks for this great addition!
I might be a bit late for the party, but I think it is more convenient to accept an AbortSignal and not an AbortController,
(which is not always available).
any chance that can change? I am willing to open a PR if that change makes sense to you

@OrKoN
Copy link
Collaborator Author

OrKoN commented May 1, 2023

That has been already addressed in #10048

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.

[Feature]: ability to cancel "waitFor" promises
3 participants