Fix for ENG-3340. Mobile browsers don't all support the web Notifications API. We should add a check for that.
Details
Details
- Reviewers
kamil inka • kuba - Commits
- rCOMM1be814d0bf66: [web] Check if notifications are supported
Check in safari on iOS emulator. Before the changes, website displays the error. After the diff it works fine.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/push-notif/push-notifs-handler.js | ||
---|---|---|
162 ↗ | (On Diff #24737) | Shouldn't this be || !supported? |