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

Revision history for this message
Gary Poster (gary) wrote :

Reviewers: mp+154403_code.launchpad.net,

Message:
Please take a look.

Description:
Add remaining incomplete components of sandbox

This adds a connection and Python juju API implementation of the
in-browser-memory environment. The final test shows a complete
integration of login from environment through to the fake backend.

This is from a spike, and I have a bit more to translate from the spike
in a follow-on branch. The follow-on branch will support deploying to
the in-memory state from the environment, and from the GUI itself. I
decided to propose this first since the branch size was getting a bit
large and it was reasonably self-sufficient as it was.

https://code.launchpad.net/~gary/juju-gui/sandboxEnv-3/+merge/154403

(do not edit description out of merge proposal)

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

Affected files:
   A [revision details]
   M app/modules-debug.js
   M app/store/env/base.js
   M app/store/env/fakebackend.js
   A app/store/env/sandbox.js
   M test/index.html
   M test/test_app.js
   M test/test_env.js
   M test/test_fakebackend.js
   A test/test_sandbox.js

« Back to merge proposal