Closed Prithpal-Sooriya closed 1 week ago
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.
Attention: Patch coverage is 0%
with 9 lines
in your changes are missing coverage. Please review.
Project coverage is 67.37%. Comparing base (
4cf7e80
) to head (ced01f5
).
Files | Patch % | Lines |
---|---|---|
...s/push-platform-notifications/services/services.ts | 0.00% | 9 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
---|---|---|---|---|---|---|---|
Chrome | Home | firstPaint | 64 | 127 | 95 | 17 | 8 |
domContentLoaded | 9 | 26 | 13 | 4 | 2 | ||
load | 51 | 2623 | 1109 | 1153 | 553 | ||
domInteractive | 9 | 26 | 13 | 4 | 2 |
Description
Push Notification API update. We now support a platform field with the registration tokens. This allows better push notification support between our different platforms.
Related issues
Fixes: N/A
Manual testing steps
This has no impact. The notification feature is not released yet.
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist