{"action":"opened","issue":{"url":"https://api.github.com/repos/w3c/manifest/issues/690","repository_url":"https://api.github.com/repos/w3c/manifest","labels_url":"https://api.github.com/repos/w3c/manifest/issues/690/labels{/name}","comments_url":"https://api.github.com/repos/w3c/manifest/issues/690/comments","events_url":"https://api.github.com/repos/w3c/manifest/issues/690/events","html_url":"https://github.com/w3c/manifest/issues/690","id":331575883,"node_id":"MDU6SXNzdWUzMzE1NzU4ODM=","number":690,"title":"Manifast and Service Worker updateViaCache","user":{"login":"ayumihamsaki","id":17784082,"node_id":"MDQ6VXNlcjE3Nzg0MDgy","avatar_url":"https://avatars2.githubusercontent.com/u/17784082?v=4","gravatar_id":"","url":"https://api.github.com/users/ayumihamsaki","html_url":"https://github.com/ayumihamsaki","followers_url":"https://api.github.com/users/ayumihamsaki/followers","following_url":"https://api.github.com/users/ayumihamsaki/following{/other_user}","gists_url":"https://api.github.com/users/ayumihamsaki/gists{/gist_id}","starred_url":"https://api.github.com/users/ayumihamsaki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ayumihamsaki/subscriptions","organizations_url":"https://api.github.com/users/ayumihamsaki/orgs","repos_url":"https://api.github.com/users/ayumihamsaki/repos","events_url":"https://api.github.com/users/ayumihamsaki/events{/privacy}","received_events_url":"https://api.github.com/users/ayumihamsaki/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-06-12T12:56:41Z","updated_at":"2018-06-12T12:56:41Z","closed_at":null,"author_association":"NONE","body":"Hello,\r\n\r\nI would just like to double check something.\r\n\r\nIn the Manifest Spec found here: https://www.w3.org/TR/appmanifest/#update_via_cache-member\r\n\r\nYou write the following:\r\n\r\n`\"update_via_cache\": \"imports\"`\r\n\r\nIn the Service Worker Spec found here: https://w3c.github.io/ServiceWorker/#enumdef-serviceworkerupdateviacache\r\n\r\nThey write the following:\r\n\r\n`\"updateViaCache\": \"imports\"`\r\n\r\nSurely both should match exactly the same to avoid confusion and possible syntax errors?\r\n\r\nGoing to add some extra links for info:\r\n\r\nGoogle's use of it: https://developers.google.com/web/updates/2018/06/fresher-sw\r\n\r\nNot sure if this issue is related: https://github.com/w3c/manifest/issues/641\r\n\r\nI just wondered if writing a manifest like this would be correct:\r\n\r\n```\r\n\"serviceworker\": {\r\n      \"src\": \"/sw.js\",\r\n      \"scope\": \"./\",\r\n      \"updateViaCache\": \"imports\"\r\n},\r\n```\r\n\r\nAlso I spoke to someone at Google about updateViaCache and they said that imports was set to default by every browser. Not sure if this is totally true, but maybe could add to the spec - if it is true?"}}
{"actor":{"display_login":"ayumihamsaki"}}