← Back to team overview

yellow team mailing list archive

Move all env code into single module (issue 6867072)

 

Reviewers: mp+138596_code.launchpad.net,

Message:
Please take a look.

Description:
Move all env code into single module

This branch is a small incremental step towards a refactored env view.
While not creating many improvements itself its now simpler to begin
parallel work around factoring the MegaModule (which will be gone
when this is all done) into proper modules, ServiceModule,
RelationModule,
etc.

https://code.launchpad.net/~bcsaller/juju-gui/ultra-mega/+merge/138596

(do not edit description out of merge proposal)


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

Affected files:
   A [revision details]
   M app/assets/javascripts/d3-components.js
   M app/modules-debug.js
   M app/templates/overview.handlebars
   M app/views/environment.js
   A app/views/topology/mega.js
   A app/views/topology/panzoom.js
   A app/views/topology/relation.js
   A app/views/topology/service.js
   A app/views/topology/topology.js
   A app/views/topology/viewport.js
   M app/views/utils.js
   M bin/lint-yuidoc
   M package.json
   M test/index.html
   M test/test_application_notifications.js
   M test/test_environment_view.js
   A test/test_topology.js
   M undocumented



-- 
https://code.launchpad.net/~bcsaller/juju-gui/ultra-mega/+merge/138596
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~bcsaller/juju-gui/ultra-mega into lp:juju-gui.


References