Code review comment for lp:~makyo/juju-gui/ambiguous-relations

Revision history for this message
Madison Scott-Clary (makyo) wrote :

Reviewers: mp+130616_code.launchpad.net,

Message:
Please take a look.

Description:
Implement ambiguous relation menu for adding rels

Displays a menu of choices when adding a relation between two services
would result in ambiguity, otherwise simply adds a relation. Also, a
bit of minor test clean-up in charm config, due to floating point
inequality.

https://code.launchpad.net/~makyo/juju-gui/ambiguous-relations/+merge/130616

(do not edit description out of merge proposal)

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

Affected files:
   A [revision details]
   M app/templates/overview.handlebars
   M app/views/environment.js
   M lib/views/stylesheet.less
   M test/test_application_notifications.js
   M test/test_charm_configuration.js
   M test/test_environment_view.js

« Back to merge proposal