ng2-cable
ng2-cable copied to clipboard
Add support for Angular 6 (rxjs 6)
Rxjs has now a new version, and subject is now imported directly from 'rxjs' Operators, like map, filter, switch map can be imported from 'rxjs/operator'
Problem is: even fixing imports, broadcaster.js fails on 'return this._eventBus.asObservable().filter(...)'
Error is:
this._eventBus.asObservable(...).filter is not a function
at Broadcaster.push../node_modules/ng2-cable/broadcaster.js.Broadcaster.on
I've made a PR to fix it https://github.com/viktor-shmigol/ng2-cable/pull/20 Actually, I think it would be better to be on a branch or separate repository, like 'ng6-cable'
@ArielAleksandrus I use angular 6.0.0 and rxjs 6.1.0. I do not have any issue at all. Here is my packages list:
@angular-devkit/build-angular 0.6.0
@angular/animations 6.0.0
@angular/cli 6.0.0
@angular/common 6.0.0
@angular/compiler 6.0.0
@angular/compiler-cli 6.0.0
@angular/core 6.0.0
@angular/forms 6.0.0
@angular/http 6.0.0
@angular/platform-browser 6.0.0
@angular/platform-browser-dynamic 6.0.0
@angular/router 6.0.0
core-js 2.5.5
rxjs 6.1.0
rxjs-compat 6.1.0
ts-node 6.0.3
typescript 2.7.2
Also I am going to replace "brodcaster" with https://github.com/viktor-shmigol/ngx-global-events. So, there is no sence to merge your PR.
@viktor-shmigol do you have a timeframe for these changes? The peer dependencies are interfering with updating to the latest angular versions.