Merge lp://staging/~elachuni/ubuntu-webcatalog/blank-apps into lp://staging/ubuntu-webcatalog

Proposed by Anthony Lenton
Status: Rejected
Rejected by: Natalia Bidart
Proposed branch: lp://staging/~elachuni/ubuntu-webcatalog/blank-apps
Merge into: lp://staging/ubuntu-webcatalog
Diff against target: 12 lines (+1/-1)
1 file modified
src/webcatalog/models/applications.py (+1/-1)
To merge this branch: bzr merge lp://staging/~elachuni/ubuntu-webcatalog/blank-apps
Reviewer Review Type Date Requested Status
Michael Nelson (community) Approve
Review via email: mp+154810@code.staging.launchpad.net

Description of the change

A trivial branch to add a blank=True argument to ApplicationWidget's applications field. This allows you to edit widgets from the admin UI, create them without apps before a bundle starts, add / remove widgets from the admin UI and clear all apps when when they're no longer needed.

To post a comment you must log in.
Revision history for this message
Michael Nelson (michael.nelson) wrote :

+1 - I'm assuming the change doesn't cause south to expect a migration due to a differing field def.

review: Approve
Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Setting MP as Rejected to reduce numbers of landing candidates in @reviewlist.
Change status again if this MP is still current.

Unmerged revisions

179. By Anthony Lenton

Fixed admin UI form so that widgets can be created without linked apps.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches