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

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

Reviewers: mp+159271_code.launchpad.net,

Message:
Please take a look.

Description:
gallimaufry

Title is in homage to benji's potpourri.

- Fix 1169350: sandbox was not marking subordinate charms correctly.
- Fix 1169668: subordinate relations were not shown because of the id
change.
- Simplify test charm store code.

https://code.launchpad.net/~gary/juju-gui/bug1169350/+merge/159271

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/8812043/

Affected files:
   A [revision details]
   M app/store/env/fakebackend.js
   M app/views/topology/relation.js
   M test/test_app.js
   M test/test_fakebackend.js
   M test/test_sandbox.js
   M test/utils.js

« Back to merge proposal