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

Foreign fetch was removed from the SW spec. #6207

Merged
merged 1 commit into from
Jun 1, 2018
Merged

Conversation

jeffposnick
Copy link
Contributor

As per w3c/ServiceWorker#1188, foreign fetch was officially removed from the service worker specification.

Target Live Date: 2018-06-07

  • This has been reviewed and approved by (@petele)
  • I have run gulp test locally and all tests pass.
  • I have added the appropriate type-something label.
  • I've staged the site and manually verified that my content displays correctly.

CC: @petele

@WebFundBot
Copy link

Whoops!

There were 59 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

WARNINGS
src/content/en/updates/2016/09/foreign-fetch.md#L243 - Updates post is missing RSS feed widget: '{% include "web/_shared/rss-widget-updates.html" %}'
src/content/en/updates/2016/09/foreign-fetch.md#L9 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L11 - Common typo found: 'Service Worker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L17 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L24 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L71 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L79 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L81 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L81 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L92 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L234 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L236 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/updates/2016/09/foreign-fetch.md#L24 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L26 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L28 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L30 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L36 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L42 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L48 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L50 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L54 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L58 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L62 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L71 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L73 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L79 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L81 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L82 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L83 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L85 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L87 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L94 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L98 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L102 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L108 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L114 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L116 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L129 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L130 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L134 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L136 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L138 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L160 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L162 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L163 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L164 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L166 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L183 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L187 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L189 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L191 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L202 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L217 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L221 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L225 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L228 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L229 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L232 - Line must be at most 100 characters
src/content/en/updates/2016/09/foreign-fetch.md#L242 - Line must be at most 100 characters

@petele petele merged commit 2e059fb into master Jun 1, 2018
@petele petele deleted the foreign-fetch-warning branch June 1, 2018 14:28
@WebFundBot
Copy link

🎉 This has been pushed live to https://developers.google.com/web/

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