Code review comment for lp://staging/~gary/juju-gui/bug1247903

Revision history for this message
Gary Poster (gary) wrote :

*** Submitted:

Re-enable Landscape sim and fix inspector link

Changes to move units to service attributes broke the Landscape
simulation because they were buggy, and we had no tests. Fixing that
revealed that Landscape links were broken in the inspector if units
appeared with Landscape problems while you observed the inspector. This
branch also fixes that problem.

To QA, run the simulator on the sandbox and create 100 or 200 units in a
service. Keep the inspector open after creation. You should see a few
Landscape issues appear within a few seconds. If you open up the
Landscape sections of the inspector, the link to Landscape at the bottom
of the section should have a href that looks vaguely like it might point
to Landscape, if Landscape were really hooked up.

R=rharding
CC=
https://codereview.appspot.com/21440044

https://codereview.appspot.com/21440044/

« Back to merge proposal