Launchpad itself

Merge lp:~jml/launchpad/remove-archive-commercial into lp:launchpad

  1. remove-archive-commercial
  2. Merge into devel
Proposed by Jonathan Lange on 2012-06-08
Status: Merged
Approved by: Jonathan Lange on 2012-06-15
Approved revision: no longer in the source branch.
Merged at revision: 15429
Proposed branch: lp:~jml/launchpad/remove-archive-commercial
Merge into: lp:launchpad
Diff against target: 0 lines
To merge this branch: bzr merge lp:~jml/launchpad/remove-archive-commercial
Reviewer Review Type Date Requested Status
Stuart Bishop 2012-06-08 Approve on 2012-06-08
Review via email: mp+109332@code.launchpad.net

Commit Message

Remove support code to handle two different column names for Archive.suppress_subscription_notifications.

Description of the Change

This branch removes the code we added to support IArchive.suppress_subscription_notifications being available as 'suppress_subscription_notifications' and as 'commercial' in the database. Henceforth, it is now just suppress_subscription_notifications. commercial is gone.

This branch may only land after the related database patch (https://code.launchpad.net/~jml/launchpad/db-rename-archive-commercial-to-suppress/+merge/109179) has landed and been applied to all database servers.

To post a comment you must log in.
Stuart Bishop (stub) wrote :

Looks good.

review: Approve

Preview Diff

Empty