Page MenuHomePhabricator

Web Push notifications
Needs ReviewPublic

Authored by duraseb on Aug 30 2015, 4:12 PM.
Tags
None
Referenced Files
F15424255: D14011.id33879.diff
Sat, Mar 22, 9:08 PM
F15391620: D14011.diff
Sat, Mar 15, 10:45 AM
F15378297: D14011.id.diff
Thu, Mar 13, 1:34 PM
Unknown Object (File)
Sun, Feb 23, 11:20 AM
Unknown Object (File)
Feb 9 2025, 8:29 AM
Unknown Object (File)
Jan 28 2025, 5:26 PM
Unknown Object (File)
Jan 27 2025, 9:16 AM
Unknown Object (File)
Jan 26 2025, 2:48 PM

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8924: Add push notification support
Summary

refs T8924

This is a proposed solution for Web Push notifications for Phabricator.
Supported apps: Conpherence

Later improvements:

  • display details in notifications instead of the generic information
  • clicking on the notification should open conpherence chat page
  • only send push notifications if a user has been inactive for some time
  • wait for Firefox support not to be that flaky
Test Plan

Diff Detail

Repository
rP Phabricator
Branch
webpush-notifications
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/applications/webpush/controller/PhabricatorManifestController.php:12TXT3Line Too Long
Warningsrc/applications/webpush/settings/PhabricatorWebpushNotificationsSettingsPanel.php:22TXT3Line Too Long
Warningsrc/applications/webpush/settings/PhabricatorWebpushNotificationsSettingsPanel.php:22XHP36Slow Construct
Warningsrc/applications/webpush/settings/PhabricatorWebpushNotificationsSettingsPanel.php:159TXT3Line Too Long
Warningsrc/applications/webpush/settings/PhabricatorWebpushNotificationsSettingsPanel.php:177TXT3Line Too Long
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:1JAVELIN5`javelinsymbols` Not In Path
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:8W117JSHintW117
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:23W117JSHintW117
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:28TXT3Line Too Long
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:29W117JSHintW117
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:31TXT3Line Too Long
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:42W117JSHintW117
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:48W117JSHintW117
Warningwebroot/rsrc/js/application/config/webpush-subscriptions.js:60W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:5W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:6W098JSHintW098
Warningwebroot/rsrc/js/core/ServiceWorker.js:6W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:7W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:8TXT3Line Too Long
Warningwebroot/rsrc/js/core/ServiceWorker.js:14W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:19W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:21W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:22W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:26W117JSHintW117
Warningwebroot/rsrc/js/core/ServiceWorker.js:30W117JSHintW117
Unit
Tests Passed
Build Status
Buildable 7846
Build 8723: [Placeholder Plan] Wait for 30 Seconds
Build 8722: arc lint + arc unit