- From: Arthur Stolyar <notifications@github.com>
- Date: Sat, 25 Jun 2016 09:49:23 -0700
- To: slightlyoff/ServiceWorker <ServiceWorker@noreply.github.com>
- Cc:
Received on Saturday, 25 June 2016 16:49:52 UTC
Yes, seems correct. But also add `.catch(...)` to the end of `caches.match(...)` because older versions of browsers reject that promise if cache doesn't exist (_new versions should just fulfill with `undefined`_). So it should be like this: ```js return caches.match(request, { cacheName: 'special-assets' }).catch(function() {}); ``` --- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/slightlyoff/ServiceWorker/issues/917#issuecomment-228556596
Received on Saturday, 25 June 2016 16:49:52 UTC