Code review comment for lp:~gary/juju-gui/bug1090716

Revision history for this message
Nicola Larosa (teknico) wrote :

Land as is.

Very nice, thanks for completing this.

https://codereview.appspot.com/7069068/diff/1/app/views/utils.js
File app/views/utils.js (right):

https://codereview.appspot.com/7069068/diff/1/app/views/utils.js#newcode666
app/views/utils.js:666: return charmUrl &&
utils.isGuiCharmUrl(charmUrl);
So it's ok that this can happen, good. Yesterday I was not sure about
it.

https://codereview.appspot.com/7069068/

« Back to merge proposal