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

Use recommended rxjs import form #49

Open
ddohler opened this issue Feb 21, 2019 · 1 comment
Open

Use recommended rxjs import form #49

ddohler opened this issue Feb 21, 2019 · 1 comment

Comments

@ddohler
Copy link
Contributor

ddohler commented Feb 21, 2019

Importing 'rxjs/Rx' or 'rxjs' is not recommended. We should switch all imports to use the recommended form, which is import { Observable } from 'rxjs/Observable ';.

@ddohler
Copy link
Contributor Author

ddohler commented Mar 29, 2019

Doing this may cause problems in the Lab and Temperate; the improper imports in this project seem to be enabling functionality that wouldn't otherwise work in those projects. That's not a blocker, but all three will need to be tested.

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