Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(9376)

Issue 7214047: GUI should handle reconnects gracefully.

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 3 months ago by matthew.scott
Modified:
11 years, 3 months ago
Reviewers:
mp+145022
Visibility:
Public.

Description

GUI should handle reconnects gracefully. Previously, the GUI would lose its mind on a reconnect, due to the addition of authentication and not receiving a delta. This has been fixed, and the existing test for connection behavior modified to reflect changes. https://code.launchpad.net/~makyo/juju-gui/gui-loses-its-mind-1098591/+merge/145022 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : GUI should handle reconnects gracefully. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -1 line) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M app/app.js View 1 1 chunk +4 lines, -1 line 0 comments Download
M test/test_app.js View 1 2 chunks +7 lines, -0 lines 0 comments Download

Messages

Total messages: 4
matthew.scott
Please take a look.
11 years, 3 months ago (2013-01-25 20:39:05 UTC) #1
gary.poster
Land as is. Works for me! Thank you. Gary
11 years, 3 months ago (2013-01-26 03:11:41 UTC) #2
frankban
Land as is. Thanks Matthew!
11 years, 3 months ago (2013-01-28 09:05:39 UTC) #3
matthew.scott
11 years, 3 months ago (2013-01-28 15:20:19 UTC) #4
*** Submitted:

GUI should handle reconnects gracefully.

Previously, the GUI would lose its mind on a reconnect, due to the addition of
authentication and not receiving a delta.  This has been fixed, and the existing
test for connection behavior modified to reflect changes.

R=gary.poster, frankban
CC=
https://codereview.appspot.com/7214047
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b