Page MenuHomePhabricator

Show Aphlict connection status in notification menu
ClosedPublic

Authored by epriestley on Jun 23 2014, 9:48 PM.
Tags
None
Referenced Files
F15425168: D9700.diff
Sun, Mar 23, 2:54 AM
F15420567: D9700.id.diff
Fri, Mar 21, 3:01 PM
F15408484: D9700.id23293.diff
Tue, Mar 18, 11:28 PM
F15396096: D9700.id.diff
Sun, Mar 16, 11:50 AM
F15388452: D9700.id23293.diff
Sat, Mar 15, 3:50 AM
F15345121: D9700.id.diff
Mon, Mar 10, 8:39 AM
F15286935: D9700.id.diff
Tue, Mar 4, 7:31 PM
Unknown Object (File)
Thu, Feb 27, 10:15 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).