-
Notifications
You must be signed in to change notification settings - Fork 89
Plan to use Rx5 instead of Rx4 due to breaking changes? #122
Comments
I know this is frowned upon, but +1. |
Hey @nmn! I'm still very new to RxJS, so could you elaborate on why this is frowned upon :)? Thanks a lot! |
@tihuan I was talking about the "+1" comments on github. |
@nmn Ahaha, I see! I didn't know that's a thing! TIL 🎅 |
There are actually a bunch of api changes between rx 4 and 5 now: https://github.com/ReactiveX/rxjs/blob/master/MIGRATION.md |
I just created #146 which contains the changes to make rx.angular.js work with RxJS 5 (beta 8 - but I expect the changes will continue to work future betas as RxJS 5 closes on RC the incompatible changes should be fewer) |
Woohoo! Thanks for the fabulous work 💯 ! |
You the man @awk! |
Is this still on the cards ? |
@mickdelaney I hope so! |
I need this too please! |
Rx5 with Angular 1 ? note: There's some discussion on this issue in the PR #146 |
I saw that one, should have paid more attention to it, thanks. |
Since Rx5 introduces some breaking changes (link), I was wondering if
rx.angular.js
will eventually move to Rx5 too.For example,
rx.Observable.replay()
is nowrx.Observable.publishReplay()
in Rx5.P.S. I'm using
rx.angular.js
in an ng1.4.8 app :)Thanks a lot!
The text was updated successfully, but these errors were encountered: