toStream should produce an iterable, for better compatiblity with other libs

[phiresky]

Just spent some time trying to get it to work with the npm package rx but it didn't work. I only noticed I had to use RxJS 5 (npm package rxjs) when I looked at the relevant tests file.
With RxJS 5 the examples work fine.

The errors were:
(toStream)

Uncaught TypeError: Object is not iterable
    at getIterable (rx.all.js:2706)
    at FromObservable.subscribeCore (rx.all.js:2634)
    at FromObservable.tryCatcher (rx.all.js:63)
    at setDisposable (rx.all.js:2082)

(fromStream)

Cannot read property fn of undefined
(forgot to save stack trace)

[mweststrate]

@phiresky the goal is to be compatible with any stream library through Symbol.observable. Did you test it in an environment that supports Symbols without accidentally compiling / polyfilling down to ES5?

[mweststrate]

Ah probably rx itself doesn't support Symbol.observable yet? Adding support for Symbol.iterable as well around here will probably fix the issue

[mweststrate]

Closed for inactivity

[hudzenko]

Smth related I think...
benlesh/symbol-observable#38
In angular I had to add polyfill:
import 'core-js/features/symbol/observable';
As mentioned above Symbol.observable is not supported for some reason in rxjs