~bac/charmworld/bug-1379397

Viewing all changes in revision 496.

  • Committer: Tarmac
  • Author(s): Brad Crittenden
  • Date: 2014-03-20 20:22:02 UTC
  • mfrom: (491.2.18 remove-bundles)
  • Revision ID: tarmac-20140320202202-syqt88y73u4d5rik
Add to GUI the ability to delete charms & bundles.

On the charm detail page, a 'Delete' link is added if the user has edit
permission.

On the bundle detail page, a new 'Actions' section is added that has 'Feature'
and 'Delete' links.

We talked about removing the charmworld/scripts/remove_charm.py file but that
isn't done in this branch.

QA: ingest some charms and bundles.  While not logged in, visit the charm and
bundle pages and notice you don't see the 'Delete' links.

Log in.

Now see the links.  Click 'Delete' then cancel or submit without checking the
box.  Note the charm isn't deleted.  Now delete it for real.  Note that it is
gone.  

Repeat for bundles.

Try featuring a bundle.

Late breaking thought:  deleting a charm that a bundle depends upon may cause
it to be undeployable.  Refuse to delete a charm if a bundle depends on it or
just leave it be?  Note this is no worse than the remove_charm.py script,
though it is easier to do.

https://codereview.appspot.com/78220043/

R=benji, makyo, rharding.

Approved by Juju Gui Bot.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: