-
Notifications
You must be signed in to change notification settings - Fork 3k
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
race typings broken for mixed observable types in 6.4.0 #4642
Labels
bug
Confirmed bug
Comments
benlesh
added a commit
to benlesh/rxjs
that referenced
this issue
Mar 15, 2019
- Actually accept other `ObservableInput` types (Promises, etc) - Fixes issue where TypeScript would complain when you tried to subscribe to a race between to Observable types. Fixes ReactiveX#4390 Fixes ReactiveX#4642
benlesh
added a commit
that referenced
this issue
Mar 15, 2019
BioPhoton
pushed a commit
to BioPhoton/rxjs
that referenced
this issue
May 15, 2019
- Actually accept other `ObservableInput` types (Promises, etc) - Fixes issue where TypeScript would complain when you tried to subscribe to a race between to Observable types. Fixes ReactiveX#4390 Fixes ReactiveX#4642
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Bug Report
Current Behavior
Subscribing to the result of a
race
of two different types of observables results in a type error.Reproduction
stackblitz
Expected behavior
It shouldn't error.
Environment
Possible Solution
Fix the types to not be
Observable<string> | Observable<number>
and beObservable<string | number>
The text was updated successfully, but these errors were encountered: