Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace HttpModule with HttpClientModule #536

Closed
cppwfs opened this issue Nov 7, 2017 · 3 comments · Fixed by #859
Closed

Replace HttpModule with HttpClientModule #536

cppwfs opened this issue Nov 7, 2017 · 3 comments · Fixed by #859
Assignees
Labels
type/technical-debt Techical Dept
Milestone

Comments

@cppwfs
Copy link
Contributor

cppwfs commented Nov 7, 2017

After migrating to Angular 5 we need to replace HttpModule with HttpClientModule. Because HttpModule will be deprecated.
https://blog.angular.io/version-5-0-0-of-angular-now-available-37e414935ced

@sabbyanandan
Copy link
Contributor

@ghillert: Is this already covered in the Angular5 PR? Perhaps close this as a duplicate if so?

@sabbyanandan
Copy link
Contributor

@ghillert: Thoughts?

@ghillert
Copy link
Contributor

Due to #832 we should migrate to the HttpClientModule as soon as possible.

@sabbyanandan sabbyanandan added this to the 1.6.0.RC1 milestone Jun 27, 2018
@sabbyanandan sabbyanandan added the type/technical-debt Techical Dept label Jul 16, 2018
@sabbyanandan sabbyanandan modified the milestones: 1.6.0.RC1, 1.7.0.M1 Jul 20, 2018
oodamien pushed a commit that referenced this issue Jul 27, 2018
* Update Tests (Refactoring of HttpClient mocks)
* Ensure that lint warnings related to Http module are fixed
* Minor fix to make bundle analyzer work: Update `webpack-bundle-analyzer` to `2.13.1`
* Fix RxJS lint warnings due to deprecation
* Start to Remove unused imports
* Fix Lint errors
@oodamien oodamien removed the in pr label Jul 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/technical-debt Techical Dept
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants