-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(notification): add service worker support for @ng-web-apis/notification
#218
base: main
Are you sure you want to change the base?
Conversation
Visit the preview URL for this PR (updated for commit 8540266): https://ng-web-apis-common--pr218-notification-api-for-rg2goj9u.web.app (expires Sat, 16 Dec 2023 11:25:40 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: a9d81cd8541a1a45fa12e730c551222bb3d5ceb9 |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #218 +/- ##
==========================================
- Coverage 88.84% 87.31% -1.53%
==========================================
Files 204 208 +4
Lines 1210 1246 +36
Branches 130 134 +4
==========================================
+ Hits 1075 1088 +13
- Misses 109 132 +23
Partials 26 26
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
a065191
to
c79b71d
Compare
a611f17
to
c683182
Compare
c683182
to
b16edfa
Compare
b16edfa
to
8540266
Compare
PR Type
What kind of change does this PR introduce?
What is the new behaviour?
Partially solves #96
Add support for mobile devices
IOS: