LGTM
https://codereview.appspot.com/6718048/diff/1/app/views/environment.js File app/views/environment.js (right): https://codereview.appspot.com/6718048/diff/1/app/views/environment.js#newcode784 app/views/environment.js:784: }); This looks fine to me, can we however be sure those are the only three states bubbling to this layer? I know that is supposed to be the case, but should these ever come out of sync having a fallback value here might make sense. Probably fine for now though https://codereview.appspot.com/6718048/ -- https://code.launchpad.net/~makyo/juju-gui/consistent-health-graph/+merge/130005 Your team Juju GUI Hackers is requested to review the proposed merge of lp:~makyo/juju-gui/consistent-health-graph into lp:juju-gui. -- Mailing list: https://launchpad.net/~yellow Post to : [email protected] Unsubscribe : https://launchpad.net/~yellow More help : https://help.launchpad.net/ListHelp
