Page MenuHomePhabricator

Allow the notifications from Aphlict debug messages to expire.
ClosedPublic

Authored by joshuaspence on May 30 2014, 1:40 PM.
Tags
None
Referenced Files
F15396209: D9339.diff
Mar 16 2025, 12:39 PM
Unknown Object (File)
Feb 23 2025, 6:18 PM
Unknown Object (File)
Feb 9 2025, 1:16 PM
Unknown Object (File)
Feb 1 2025, 1:10 PM
Unknown Object (File)
Jan 27 2025, 5:35 AM
Unknown Object (File)
Jan 27 2025, 5:26 AM
Unknown Object (File)
Jan 21 2025, 9:05 AM
Unknown Object (File)
Jan 17 2025, 7:11 PM
Subscribers

Details

Summary

Currently, debugging (log) messages from the Aphlict client trigger a visible notification in the web UI (when notification.debug is enabled). After D9327, the log messages can be quite verbose and seem to "block" legitimate notifications because they are not automatically dismissed.

Test Plan

Enabled notification.debug and observed that the debugging notifications expired and closed after a few seconds.

Diff Detail

Repository
rP Phabricator
Branch
aphlict-timeout
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 794
Build 794: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Allow the notifications from Aphlict debug messages to expire..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

Haha, I hit this too.

This revision is now accepted and ready to land.May 30 2014, 4:47 PM
epriestley updated this revision to Diff 22248.

Closed by commit rP323c529c3bac (authored by @joshuaspence, committed by @epriestley).