We need to register a service worker to get push notifs from the browser. This diff registers the service worker from the endpoint exposed by the keyserver in one of the previous diffs on page load. For now this service worker only logs when installing and activating.
Details
Details
- check if service worker is correctly registered (you can check that in the browser's dev tools)
- check that the service worker is correctly updated after changing the service worker file
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
Confused why this needs to be a React component
| web/push-notif/push-notifs-handler.js | ||
|---|---|---|
| 7 ↗ | (On Diff #22920) | Is there a reason this needs to be in a React component? What if we just called this from the global scope? |
| 16 ↗ | (On Diff #22920) | Just wondering... what does ESLint think is missing here? Based on my intuition about this ESLint rule, I would guess it thinks navigator and electron are both global/imported so don't need to be in the dep list |
Comment Actions
Moved eslint annotation to a later diff.
| web/push-notif/push-notifs-handler.js | ||
|---|---|---|
| 7 ↗ | (On Diff #22920) | While right now it's just using global objects, in the later diffs there are react hooks added inside this component. |
| 16 ↗ | (On Diff #22920) | Sorry, I just forgot to put this in the correct diff when splitting this into multiple diffs. I'm going to move it the diffs that need this annotation. |