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

Fix workbox-broadcast-update link #2212

Closed
wants to merge 1 commit into from
Closed

Fix workbox-broadcast-update link #2212

wants to merge 1 commit into from

Conversation

abraham
Copy link

@abraham abraham commented Sep 2, 2019

R: @jeffposnick @philipwalton

Current URL 404s.

@philipwalton
Copy link
Member

Hmmm, I think the fix for the problem here is our docs site needs to be updated. Specifically the file name of this file as well as some of its titles. @abraham if you're able to make that update and cc me, I can approve and merge.

We also have a TODO to update some of the content there in line with v5 changes: #2122

@philipwalton
Copy link
Member

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.314% when pulling 3f8cce3 on abraham:patch-1 into ae80e39 on GoogleChrome:master.

@abraham
Copy link
Author

abraham commented Sep 2, 2019

@abraham abraham closed this Sep 2, 2019
@abraham abraham deleted the patch-1 branch September 2, 2019 19:50
@philipwalton
Copy link
Member

I've published the demos, and the page on developers.google.com/web/ should be updated too, so everything should work now. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants