[TS Typing] allow mergeAll on Observable<T[]> like mergeMap(x => x) allows

[Dorus]

On an observable of an array, it's possible to call .mergeMap(x => x) instead of .mergeMap(x => Rx.Observalble.from(x)).

While .mergeMap(x => x) should be identical to mergeAll(), in TS it's not allowed to call mergeAll() on Observable<T[]>, but .mergeMap(x => x) is allowed.

[kwonoj]

I'm not sure if I understood issue correctly. Can you provide detail with snippet example for clarity?

[Dorus]

Code sample:

Rx.Observable.of([1,2,3]).mergeMap(x => x)

gives:

1
2
3

and

Rx.Observable.of([1,2,3]).mergeAll()

gives

[ts] Property 'subscribe' does not exist on type 'number[]'.

This should produce the same result, an observable of 1,2,3, but in Ts, the second one wont compile.

[kwonoj]

Maybe I'm lost, but isn't this in above code snippet?

[kwonoj]

Oh, now I see. Sorry for confusions.

[Dorus]

@kwonoj Sorry for the confusion, i should have added subscribe to the observable. My point is that the second line will return number[] instead of Observable<number>

[kwonoj]

Actually I was surprised we have updated concatAll for similar reason but forgot mergemap. #1436 :p think you can blame me for that.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.