Code review comment for lp:~rharding/juju-gui/bundle-counts

Revision history for this message
Richard Harding (rharding) wrote :

Reviewers: mp+196135_code.launchpad.net,

Message:
Please take a look.

Description:
Display bundle download counts in the UI.

- Update the model to carry the attribute from the api.
- Update the token to display the "Deployed X times".
- Update the details to also display the count.
- Drive-by to update the attributes in the charm model.
- Update tests with updated api download from the server that includes
the
download counts. Required updating tests to the new data.

https://code.launchpad.net/~rharding/juju-gui/bundle-counts/+merge/196135

(do not edit description out of merge proposal)

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

Affected files (+1306, -1168 lines):
   A [revision details]
   M app/models/bundle.js
   M app/models/charm.js
   M app/templates/bundle-token.handlebars
   M app/templates/bundle.handlebars
   M test/data/browserbundle.json
   M test/test_bundle_details_view.js
   M test/test_model_bundle.js
   M test/test_token.js
   M test/test_utils.js

« Back to merge proposal