beverloo / periodic-background-sync

Periodic Background Sync Explainer
23 stars 1 forks source link

Explainer feedback #10

Closed jakearchibald closed 5 years ago

jakearchibald commented 5 years ago

Pushing a critical application update.

There are a few references to 'pushing' content, but isn't periodic sync "pull"?

Updated icons after a UI overhaul of a web app.

Minor thing, but it seemed odd that you'd only fetch new icons if the whole UI has been overhauled.

Fresh articles from news sites.

This is a great use-case. It might be worth pointing out why push isn't suited to this. The updates will be too frequent, so the content site will have to batch pushes, but the browser is in a much better position to do that.

Additional tasks like resource management add complexity to this approach

From reading this, I'm not sure what resources it's referring to, or who would be managing them.

Enable a web app to run tasks periodically on network connectivity

"on network connectivity" makes it sound like a triggering event. Maybe "Enable a web app to run tasks periodically while the device has network connectivity".

Removing a periodic sync when the user signs out

Maybe don't mentioning signing out. Clear-Site-Data is a much more robust feature for that use case.

Should we mention that UAs could/should be stricter on how long the service worker stays awake for this feature compared to more visible things like push?