Code review comment for lp:~bac/juju-gui/1096230

Revision history for this message
Brad Crittenden (bac) wrote :

Reviewers: mp+142513_code.launchpad.net,

Message:
Please take a look.

Description:
Add support for _annotation calls to env.

The calls update_annotations, get_annotations, and remove_annotations
are
added to the env object.

As a drive-by, in test_env.js the unnecessary use of async calls was
eliminated by removing the 'done' parameter. Leaving it in place where
not
required was confusing.

https://code.launchpad.net/~bac/juju-gui/1096230/+merge/142513

(do not edit description out of merge proposal)

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

Affected files:
   A [revision details]
   M app/store/env.js
   M test/test_env.js

« Back to merge proposal