Page MenuHomePhabricator

[web] Check if notifications are supported
ClosedPublic

Authored by michal on Apr 6 2023, 8:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 27, 9:55 PM
Unknown Object (File)
Fri, Dec 27, 9:55 PM
Unknown Object (File)
Fri, Dec 27, 9:55 PM
Unknown Object (File)
Fri, Dec 27, 9:55 PM
Unknown Object (File)
Fri, Dec 27, 9:52 PM
Unknown Object (File)
Sat, Dec 21, 8:09 AM
Unknown Object (File)
Sat, Dec 21, 7:06 AM
Unknown Object (File)
Sat, Dec 21, 3:15 AM
Subscribers

Details

Summary

Fix for ENG-3340. Mobile browsers don't all support the web Notifications API. We should add a check for that.

Test Plan

Check in safari on iOS emulator. Before the changes, website displays the error. After the diff it works fine.

Diff Detail

Repository
rCOMM Comm
Branch
michal/stuff1
Lint
No Lint Coverage
Unit
No Test Coverage