Merge lp:~jskrzeszewska/mailman/enablingArchivers into lp:mailman
Proposed by
Joanna Skrzeszewska
Status: | Merged |
---|---|
Merge reported by: | Barry Warsaw |
Merged at revision: | not available |
Proposed branch: | lp:~jskrzeszewska/mailman/enablingArchivers |
Merge into: | lp:mailman |
Diff against target: |
279 lines (+137/-19) 5 files modified
src/mailman/interfaces/mailinglist.py (+11/-0) src/mailman/model/mailinglist.py (+55/-4) src/mailman/rest/configuration.py (+28/-1) src/mailman/rest/tests/test_lists.py (+26/-0) src/mailman/runners/archive.py (+17/-14) |
To merge this branch: | bzr merge lp:~jskrzeszewska/mailman/enablingArchivers |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Barry Warsaw | Approve | ||
Nicki Hutchens (community) | Approve | ||
Nicki Hutchens | Pending | ||
Review via email: mp+184461@code.launchpad.net |
Description of the change
Changes for enabling/disabling archivers using postorius (bug 1158040).
To post a comment you must log in.
Hi Joanna,
Apologies for the long delay, but I've finally managed to review this branch. I decided to make a number of changes to the semantics and API, but your branch was instrumental in getting this new feature landed. Thanks very much for your contribution!
The main change that I made was to the REST API. Rather than put the list of archivers inside the list's 'configuration' resource, I decided to make it a separate sub-resource of the mailing list. This helped map the semantics of getting the list of enabled archivers, and changing their state, better to HTTP commands.
r7229 in trunk contains the motified and merged branch.