Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

move CRViz to RxJS 6 #72

Closed
rashley-iqt opened this issue Jun 7, 2018 · 0 comments
Closed

move CRViz to RxJS 6 #72

rashley-iqt opened this issue Jun 7, 2018 · 0 comments
Assignees

Comments

@rashley-iqt
Copy link
Member

RxJS 6 has discarded operator chaining in favor of operator piping
(for good reason see: https://github.com/ReactiveX/rxjs/blob/91088dae1df097be2370c73300ffa11b27fd0100/doc/pipeable-operators.md for more detail)
needless to say this presents several breaking changes for CRViz in that it breaks ALL of our epics. In the interests of smaller more performant js bundles and objects and eliminating unecessary dependencies we should upgade to RxJS 6.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant