Deprecate this package - Githubissues

Financial-Times / n-service-worker

❌ DECOMMISSIONED Global service worker component for next.ft.com

5 stars 2 forks source link

Deprecate this package #166

Open i-like-robots opened 4 years ago

i-like-robots commented 4 years ago

Actions:

[ ] Deploy code to unregister service worker from visitor's browsers
[ ] Shut down service worker service (https://biz-ops.in.ft.com/System/next-sw)
[ ] Clean up dependencies on this package
[ ] Remove entry from the service registry
[ ] Archive repository
[ ] Deprecate npm package

thurstontye commented 4 years ago

@i-like-robots actually myFT push notifications are still active, can we have a chat about this early next week. Sorry.

i-like-robots commented 4 years ago

That was a last minute reprieve!

thurstontye commented 4 years ago

@i-like-robots completely forgot to talk to you about this. My suggestion would be to set up a new service worker just for the notification feature. I'm not sure what the best practice is on having multiple service workers is. Probably not the best of ideas but would allow us to decom this one.