Repository navigation
feat(aio): Convert AIO to use the new Service Worker 5.0.0. #19795
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
Changes from all commits
5ddc010
803d169
7e9e472
b5291c2
8f18059
6e2e11e
6ba6bc6
9f69609
6922a63
9ada37b
63eb0ad
515178b
6e41f61
8293cfe
146de97
3df2e77
c36774f
323afdf
0832d58
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,128 @@ | ||
| { | ||
| "index": "/index.html", | ||
| "assetGroups": [ | ||
| { | ||
| "name": "app-shell", | ||
| "installMode": "prefetch", | ||
| "updateMode": "prefetch", | ||
| "resources": { | ||
| "files": [ | ||
| "/index.html", | ||
| "/pwa-manifest.json", | ||
| "/app/search/search-worker.js", | ||
| "/assets/images/favicons/favicon.ico", | ||
| "/assets/js/*.js", | ||
| "/*.css", | ||
| "/*.js" | ||
| ], | ||
| "urls": [ | ||
| "https://fonts.googleapis.com/**", | ||
| "https://fonts.gstatic.com/s/**", | ||
| "https://maxcdn.bootstrapcdn.com/**" | ||
| ] | ||
| } | ||
| }, { | ||
| "name": "assets-eager", | ||
| "installMode": "prefetch", | ||
| "updateMode": "prefetch", | ||
| "resources": { | ||
| "files": [ | ||
| "/assets/images/**", | ||
| "/generated/images/marketing/**", | ||
| "!/assets/images/favicons/**", | ||
| "!/**/_unused/**" | ||
| ] | ||
| } | ||
| }, { | ||
| "name": "assets-lazy", | ||
| "installMode": "lazy", | ||
| "updateMode": "prefetch", | ||
| "resources": { | ||
| "files": [ | ||
| "/assets/images/favicons/**", | ||
| "/generated/ie-polyfills.min.js", | ||
| "!/**/_unused/**" | ||
| ] | ||
| } | ||
| }, { | ||
| "name": "docs-index", | ||
| "installMode": "prefetch", | ||
| "updateMode": "prefetch", | ||
| "resources": { | ||
| "files": [ | ||
| "/generated/*.json", | ||
| "/generated/docs/*.json", | ||
| "/generated/docs/api/api-list.json", | ||
| "/generated/docs/app/search-data.json" | ||
| ] | ||
| } | ||
| }, { | ||
| "name": "docs-lazy", | ||
| "installMode": "lazy", | ||
| "updateMode": "lazy", | ||
| "resources": { | ||
| "files": [ | ||
| "/generated/docs/**/*.json", | ||
| "/generated/images/**", | ||
| "!/**/_unused/**" | ||
| ] | ||
| } | ||
| } | ||
| ], | ||
| "navigationUrls": [ | ||
| "/**", | ||
| "!/**/*.*", | ||
| "!/**/*__*", | ||
| "!/**/*__*/**", | ||
| "!/**/stackblitz", | ||
| "!/**/stackblitz.html", | ||
| "!/api/*/**/*-*", | ||
| "!/api/**/AnimationStateDeclarationMetadata*", | ||
| "!/api/**/CORE_DIRECTIVES*", | ||
| "!/api/**/DirectiveMetadata*", | ||
| "!/api/**/HTTP_PROVIDERS*", | ||
| "!/api/**/NgFor", | ||
| "!/api/**/NgFor-*", | ||
| "!/api/**/PLATFORM_PIPES*", | ||
| "!/api/animate/**", | ||
| "!/api/api/**", | ||
| "!/api/common/Control*", | ||
| "!/api/common/MaxLengthValidator*", | ||
| "!/api/common/NgModel*", | ||
| "!/api/platform-browser/AnimationDriver*", | ||
| "!/api/testing/**", | ||
| "!/docs/?*", | ||
| "!/docs/*/**", | ||
| "!/guide/cli-quickstart", | ||
| "!/guide/cli-quickstart.html", | ||
| "!/guide/cli-quickstart/", | ||
| "!/guide/learning-angular", | ||
| "!/guide/learning-angular.html", | ||
| "!/guide/learning-angular/", | ||
| "!/guide/metadata", | ||
| "!/guide/metadata.html", | ||
| "!/guide/metadata/", | ||
| "!/guide/ngmodule", | ||
| "!/guide/ngmodule.html", | ||
| "!/guide/ngmodule/", | ||
| "!/guide/service-worker-getstart", | ||
| "!/guide/service-worker-getstart.html", | ||
| "!/guide/service-worker-getstart/", | ||
| "!/guide/service-worker-comm", | ||
| "!/guide/service-worker-comm.html", | ||
| "!/guide/service-worker-comm/", | ||
| "!/guide/service-worker-configref", | ||
| "!/guide/service-worker-configref.html", | ||
| "!/guide/service-worker-configref/", | ||
| "!/guide/webpack", | ||
| "!/guide/webpack.html", | ||
| "!/guide/webpack/", | ||
| "!/news", | ||
| "!/news.html", | ||
| "!/news/", | ||
| "!/styleguide", | ||
| "!/styleguide/**", | ||
| "!/testing", | ||
| "!/testing/**" | ||
| ] | ||
| } |
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,7 +8,7 @@ | |
| "scripts": { | ||
| "preinstall": "node ../tools/yarn/check-yarn.js", | ||
| "postinstall": "node tools/cli-patches/patch.js && uglifyjs node_modules/lunr/lunr.js -c -m -o src/assets/js/lunr.min.js --source-map", | ||
| "aio-use-local": "node tools/ng-packages-installer overwrite . --debug --ignore-packages @angular/service-worker", | ||
| "aio-use-local": "node tools/ng-packages-installer overwrite . --debug", | ||
| "aio-use-npm": "node tools/ng-packages-installer restore .", | ||
| "aio-check-local": "node tools/ng-packages-installer check .", | ||
| "ng": "yarn check-env && ng", | ||
|
|
@@ -55,51 +55,44 @@ | |
| "boilerplate:test": "node tools/examples/test.js", | ||
| "generate-stackblitz": "node ./tools/stackblitz-builder/generateStackblitz", | ||
| "generate-zips": "node ./tools/example-zipper/generateZips", | ||
| "sw-manifest": "ngu-sw-manifest --dist dist --in ngsw-manifest.json --out dist/ngsw-manifest.json", | ||
| "sw-copy": "cp node_modules/@angular/service-worker/bundles/worker-basic.min.js dist/", | ||
| "build-404-page": "node scripts/build-404-page", | ||
| "build-ie-polyfills": "yarn webpack-cli src/ie-polyfills.js -o src/generated/ie-polyfills.min.js --mode production", | ||
| "update-webdriver": "webdriver-manager update --standalone false --gecko false $CHROMEDRIVER_VERSION_ARG", | ||
| "~~check-env": "node scripts/check-environment", | ||
| "~~build": "ng build", | ||
| "post~~build": "yarn build-404-page && yarn sw-manifest && yarn sw-copy" | ||
| "post~~build": "yarn build-404-page" | ||
| }, | ||
| "engines": { | ||
| "node": ">=8.9.1 <9.0.0", | ||
| "yarn": ">=1.3.2 <2.0.0" | ||
| }, | ||
| "private": true, | ||
| "dependencies": { | ||
| "@angular/animations": "6.0.0", | ||
| "@angular/cdk": "6.0.0", | ||
| "@angular/common": "6.0.0", | ||
| "@angular/core": "6.0.0", | ||
| "@angular/elements": "6.0.0", | ||
| "@angular/forms": "6.0.0", | ||
| "@angular/http": "6.0.0", | ||
| "@angular/material": "6.0.0", | ||
| "@angular/platform-browser": "6.0.0", | ||
| "@angular/platform-browser-dynamic": "6.0.0", | ||
| "@angular/platform-server": "6.0.0", | ||
| "@angular/router": "6.0.0", | ||
| "@angular/service-worker": "^1.0.0-beta.16", | ||
| "@angular/animations": "6.1.0-rc.3", | ||
| "@angular/cdk": "6.0.2", | ||
| "@angular/common": "6.1.0-rc.3", | ||
| "@angular/core": "6.1.0-rc.3", | ||
| "@angular/elements": "6.1.0-rc.3", | ||
| "@angular/forms": "6.1.0-rc.3", | ||
| "@angular/material": "6.0.2", | ||
| "@angular/platform-browser": "6.1.0-rc.3", | ||
| "@angular/platform-browser-dynamic": "6.1.0-rc.3", | ||
| "@angular/router": "6.1.0-rc.3", | ||
| "@angular/service-worker": "6.1.0-rc.3", | ||
| "@webcomponents/custom-elements": "^1.0.8", | ||
| "classlist.js": "^1.1.20150312", | ||
| "core-js": "^2.4.1", | ||
| "jasmine": "^2.6.0", | ||
| "ng-pwa-tools": "^0.0.10", | ||
| "rxjs": "6.0.0-uncanny-rc.7", | ||
| "rxjs-compat": "6.0.0-uncanny-rc.7", | ||
| "rxjs": "6.2.2", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we should now be able to drop rxjs-compat as well. can you send a follow up PR for that plz?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm dropping it in this PR 😁 |
||
| "tslib": "^1.9.0", | ||
| "web-animations-js": "^2.2.5", | ||
| "zone.js": "^0.8.26" | ||
| }, | ||
| "devDependencies": { | ||
| "@angular-devkit/build-angular": "^0.6.0", | ||
| "@angular/cli": "^6.0.0", | ||
| "@angular/compiler": "6.0.0", | ||
| "@angular/compiler-cli": "6.0.0", | ||
| "@angular/language-service": "6.0.0", | ||
| "@angular-devkit/build-angular": "0.6.7", | ||
| "@angular/cli": "^6.1.0-rc.3", | ||
| "@angular/compiler": "6.1.0-rc.3", | ||
| "@angular/compiler-cli": "6.1.0-rc.3", | ||
| "@angular/language-service": "6.1.0-rc.3", | ||
| "@types/jasmine": "^2.5.52", | ||
| "@types/jasminewd2": "^2.0.3", | ||
| "@types/node": "~6.0.60", | ||
|
|
@@ -125,6 +118,7 @@ | |
| "http-server": "^0.9.0", | ||
| "ignore": "^3.3.3", | ||
| "image-size": "^0.5.1", | ||
| "jasmine": "^2.6.0", | ||
| "jasmine-core": "^2.8.0", | ||
| "jasmine-marbles": "^0.3.1", | ||
| "jasmine-spec-reporter": "^4.1.0", | ||
|
|
@@ -150,7 +144,7 @@ | |
| "tree-kill": "^1.1.0", | ||
| "ts-node": "^3.3.0", | ||
| "tslint": "~5.9.1", | ||
| "typescript": "~2.7.2", | ||
| "typescript": "^2.9.2", | ||
| "uglify-js": "^3.0.15", | ||
| "unist-util-filter": "^0.2.1", | ||
| "unist-util-source": "^1.0.1", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how come we no longer need to deal with favicon.ico?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We haven't been using this favicon for a while (so I deleted it).
We use another favicon instead.