Page MenuHomePhabricator

Show Aphlict connection status in notification menu
ClosedPublic

Authored by epriestley on Jun 23 2014, 9:48 PM.
Tags
None
Referenced Files
F18770320: D9700.id23293.diff
Wed, Oct 8, 11:55 AM
F18741021: D9700.id.diff
Thu, Oct 2, 7:32 AM
F18613053: D9700.id.diff
Sep 14 2025, 10:40 AM
F18557916: D9700.id.diff
Sep 8 2025, 8:11 PM
F18514833: D9700.diff
Sep 5 2025, 10:51 AM
F18390719: D9700.diff
Aug 29 2025, 7:05 AM
F18095419: D9700.id23293.diff
Aug 7 2025, 11:30 PM
F17940146: D9700.id.diff
Jul 31 2025, 3:56 AM
Subscribers

Details

Summary

Fixes T5373. Ref T5281. Several changes:

  • The marshallExceptions thing is useful if JS throws an exception when invoked from Flash, so set it. The resulting exceptions are a little odd (not escaped correctly, e.g.) but way better than nothing.
  • Put connection status in the notification menu.
  • When the connection fails, try to provide contextual help where we can.
Test Plan

Screen_Shot_2014-06-23_at_2.42.25_PM.png (1×1 px, 326 KB)

Diff Detail

Repository
rP Phabricator
Branch
aphlict3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1349
Build 1349: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Show Aphlict connection status in notification menu.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: joshuaspence.
epriestley edited edge metadata.
  • Slightly cleaner handling of error.text
  • Instead of setting status directly from the master, send it to all of the clients, then set it from there.

Test plan:

  • Opened a master, saw it go through connecting -> error.
  • Opened another window, saw it connect to the master and get the correct status.
This revision is now accepted and ready to land.Jun 23 2014, 10:53 PM
epriestley updated this revision to Diff 23293.

Closed by commit rP76cefde0b3d3 (authored by @epriestley).