- From: Luke Childs <notifications@github.com>
- Date: Thu, 08 Sep 2016 22:38:52 -0700
- To: w3c/ServiceWorker <ServiceWorker@noreply.github.com>
Received on Friday, 9 September 2016 05:39:21 UTC
I personally think that:
```js
return cachedResponse || fetch(event.request);
```
makes it a lot more obvious what's going on as opposed to:
```js
return response || fetch(event.request);
```
I had to do a double take with `response` as at first glance it looked like the request was being made twice.
You can view, comment on, or merge this pull request online at:
https://github.com/w3c/ServiceWorker/pull/976
-- Commit Summary --
* Make cache example easier to understand
-- File Changes --
M explainer.md (4)
-- Patch Links --
https://github.com/w3c/ServiceWorker/pull/976.patch
https://github.com/w3c/ServiceWorker/pull/976.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/ServiceWorker/pull/976
Received on Friday, 9 September 2016 05:39:21 UTC