Page MenuHomePhabricator

Show Aphlict connection status in notification menu
ClosedPublic

Authored by epriestley on Jun 23 2014, 9:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 9, 1:52 PM
Unknown Object (File)
Sun, Feb 9, 1:52 PM
Unknown Object (File)
Wed, Jan 29, 9:14 PM
Unknown Object (File)
Wed, Jan 22, 4:10 AM
Unknown Object (File)
Wed, Jan 22, 4:10 AM
Unknown Object (File)
Wed, Jan 22, 3:40 AM
Unknown Object (File)
Jan 21 2025, 8:52 AM
Unknown Object (File)
Jan 18 2025, 5:45 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
Lint
Lint Skipped
Unit
Tests Skipped

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).