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

delayWhen's delayDurationSelector should support ObservableInput #7046

Closed
jakovljevic-mladen opened this issue Aug 25, 2022 Discussed in #6972 · 0 comments · Fixed by #7049
Closed

delayWhen's delayDurationSelector should support ObservableInput #7046

jakovljevic-mladen opened this issue Aug 25, 2022 Discussed in #6972 · 0 comments · Fixed by #7049

Comments

@jakovljevic-mladen
Copy link
Member

Discussed in #6972

Originally posted by jollytoad May 12, 2022
Some operators such as bufferWhen, windowWhen, audit allow their selector fn to return an ObservableInput which really helps with interop with native JS features, such as Promises, Iterables/AsyncIterables etc.

But the delayWhen delayDurationSelector fn must return an Observable, it would really help if this was also ObservableInput. I'm not sure if this is intentional (I don't understand why it would be), or an oversight. I'm not sure if this is the case elsewhere, but it would certainly help if it was consistent across all selector fns.

@jakovljevic-mladen jakovljevic-mladen added AGENDA ITEM Flagged for discussion at core team meetings and removed AGENDA ITEM Flagged for discussion at core team meetings labels Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant