Skip to content
This repository has been archived by the owner on Apr 9, 2022. It is now read-only.

@angular/pwa & SW related fixes/refactorings #999

Closed
wants to merge 4 commits into from
Closed

@angular/pwa & SW related fixes/refactorings #999

wants to merge 4 commits into from

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented Jun 1, 2018

The main fixes are:

  1. fix(@ schematics/angular): correctly order added dependencies in package.json
  2. fix(@ schematics/angular): allow ServiceWorker to work with baseHref
    (Fixes ServiceWorker register generated by cli with base-href don't works angular-cli#8515.)

I suspect (1) could be re-used in more places, but I have no idea where/how 😁

@gkalpak gkalpak changed the title @angular/pwa & SW related fixes/refactorings WIP - @angular/pwa & SW related fixes/refactorings Jun 2, 2018
@gkalpak gkalpak changed the title WIP - @angular/pwa & SW related fixes/refactorings @angular/pwa & SW related fixes/refactorings Jun 2, 2018
Copy link
Member

@alxhub alxhub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NGSW parts LGTM

@alexeagle
Copy link
Contributor

Sorry @gkalpak, this got lost when we moved the code back over to the angular/angular-cli repo. Do you have time to re-base this commit on master over there?

@alexeagle alexeagle closed this Sep 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ServiceWorker register generated by cli with base-href don't works
4 participants