Add a service unit when dragging the service. #309
Conversation
/** | ||
* Pull out all deplayed changes from the changeset to display to the user. | ||
Return the URL to the service icon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's drop an XXX in here to update to use the view helper for icon locating.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
The QA steps above work as prescribed. |
QA ok 👍 |
Test PASSed. |
Thank you both for the reviews! |
Test PASSed. |
Build failed: Attempt to land pull request failed |
Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge |
Test PASSed. |
Also improved the changeset summary page + clean up. QA: - make devel; - visit `/:flags:/mv/il/`; - disable the simulator; - drag a service; - hit deploy in the status bar; - you should see the deployment summary: if the description hides, click cancel and then deploy again (there is a XXX comment about this issue); - click confirm, the service should be deployed with its unit; - in the JS console `app.db.units.size()` should return 1 and `app.db.units.item(0)` should show the unit with the expected properties; - the machine view should correctly show the unit (on machine 0). TODO: - do not deploy unplaced units; - env.placeUnit to place a unit.
Also improved the changeset summary page + clean up.
QA:
/:flags:/mv/il/
;hides, click cancel and then deploy again (there is a XXX comment
about this issue);
app.db.units.size()
should return 1 andapp.db.units.item(0)
should show the unit with the expectedproperties;
TODO: