Skip to content
This repository was archived by the owner on Dec 18, 2024. It is now read-only.

feat(sw): adds ngsw config #350

Merged
merged 3 commits into from
Jan 22, 2018
Merged

feat(sw): adds ngsw config #350

merged 3 commits into from
Jan 22, 2018

Conversation

amcdnl
Copy link
Contributor

@amcdnl amcdnl commented Dec 10, 2017

This PR uses the Angular CLI 1.6 and Angular Service Worker config to setup service workers for the docs site. Cache busting seems to work based on hashing strategies the cli generates.

Important note for when testing: SWs do not work w/ ng-serve, see: https://angular.io/guide/service-worker-getstart#serving-with-http-server

@amcdnl amcdnl self-assigned this Dec 10, 2017
Copy link
Collaborator

@andrewseguin andrewseguin left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -33,6 +33,8 @@ import {
CanActivateComponentSidenav
} from './pages/component-sidenav/component-sidenav-can-load-guard';
import {HttpClientModule} from '@angular/common/http';
import { ServiceWorkerModule } from '@angular/service-worker';
import { environment } from '../environments/environment';
Copy link
Member

Choose a reason for hiding this comment

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

Omit spaces in braces

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@andrewseguin
Copy link
Collaborator

Just finished testing the service worker, looks like it works =)

@jelbourn
Copy link
Member

@amcdnl just needs rebase now to be ready to go in

# Conflicts:
#	ngsw-manifest.json
@amcdnl
Copy link
Contributor Author

amcdnl commented Jan 21, 2018

@jelbourn - good to go.

@jelbourn jelbourn merged commit b7d033a into angular:master Jan 22, 2018
@amcdnl amcdnl deleted the service-worker branch January 26, 2018 12:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants