Enable push notifications to be processed on the client

Financial-Times / n-service-worker

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

5 stars 2 forks source link

Enable push notifications to be processed on the client #88

Closed GlynnPhillips closed 7 years ago

GlynnPhillips commented 7 years ago

For now this is pretty much just a move of files that were previosuly used for push notifications. There has been some very small changes.

One improvement I can see straight away would be to refactor the track.js to work with the src/offline/spoor.js which also exists. I wanted to leave this for now though whilst we get everything set up again.

leggsimon commented 7 years ago

@GlynnPhillips could you add some tests for this please

GlynnPhillips commented 7 years ago

@leggsimon Yeah they will come soon, I am waiting to see what will stay and what will be refactored / removed. There's also not a huge amount in the main push stuff to test as its largely just browser api's.