Skip to content

add request logs for iOS:APNs and Android:FCM#639

Open
keremoge wants to merge 2 commits into
appleboy:masterfrom
keremoge:request_log
Open

add request logs for iOS:APNs and Android:FCM#639
keremoge wants to merge 2 commits into
appleboy:masterfrom
keremoge:request_log

prevent json parse error from blocking the notification

78698fa
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Apr 5, 2026 in 0s

64.89% (-0.16%) compared to 9d47494

View this Pull Request on Codecov

64.89% (-0.16%) compared to 9d47494

Details

Codecov Report

❌ Patch coverage is 31.57895% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.89%. Comparing base (9d47494) to head (78698fa).
⚠️ Report is 306 commits behind head on master.

Files with missing lines Patch % Lines
notify/notification_hms.go 0.00% 7 Missing ⚠️
notify/notification_apns.go 40.00% 1 Missing and 2 partials ⚠️
notify/notification_fcm.go 40.00% 1 Missing and 2 partials ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #639      +/-   ##
==========================================
- Coverage   65.05%   64.89%   -0.16%     
==========================================
  Files          21       21              
  Lines        1783     1792       +9     
==========================================
+ Hits         1160     1163       +3     
- Misses        562      564       +2     
- Partials       61       65       +4

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
View more details on Codecov