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

Proposed by Jonathan Lange
Status: Merged
Approved by: Jonathan Lange
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 (community) Approve
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.
Revision history for this message
Stuart Bishop (stub) wrote :

Looks good.

review: Approve

Preview Diff

Empty