Page MenuHomePhabricator

[web] Updates to push notif service worker for unused-promise Flow lint
ClosedPublic

Authored by ashoat on Dec 4 2023, 1:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 3:33 PM
Unknown Object (File)
Mon, Nov 25, 3:54 PM
Unknown Object (File)
Nov 22 2024, 8:16 AM
Unknown Object (File)
Nov 22 2024, 2:40 AM
Unknown Object (File)
Nov 7 2024, 5:36 AM
Unknown Object (File)
Oct 24 2024, 2:13 PM
Unknown Object (File)
Oct 22 2024, 3:38 AM
Unknown Object (File)
Oct 19 2024, 3:33 AM
Subscribers

Details

Summary

Two changes here:

  1. Passing skipWaiting directly in self.addEventListener('install', skipWaiting). It looks like the types work for this.
  2. Modifying the NotificationEvent.waitUntil call to wait until openWindow completes. I think the unused-promise Lint was helpful here in catching a case we should await, but would appreciate close review.

Depends on D10164

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable