[Merge] lp:~postfuturist/mailman/rest-api-delivery_mode into lp:mailman

Stephen A. Goss steveth45 at gmail.com
Thu Aug 25 02:14:49 CEST 2011


Stephen A. Goss has proposed merging lp:~postfuturist/mailman/rest-api-delivery_mode into lp:mailman.

Requested reviews:
  Mailman Coders (mailman-coders)
Related bugs:
  Bug #833132 in GNU Mailman: "Unable to query or change member delivery_mode through REST API."
  https://bugs.launchpad.net/mailman/+bug/833132
  Bug #833376 in GNU Mailman: "Attempting to PATCH an invalid attribute on a member object in REST API results in 500, instead of 400."
  https://bugs.launchpad.net/mailman/+bug/833376

For more details, see:
https://code.launchpad.net/~postfuturist/mailman/rest-api-delivery_mode/+merge/72813

Currently, in the REST API for members/subscriptions you can create a subscription with a specific delivery_mode (regular or various digest modes). However, you can't get this information back from the REST API or change it. This branch addresses these shortcomings by adding delivery_mode to the information returned by the API when GETing member info, it also allows you to PATCH the delivery_mode of a member. It handles edge/error cases, like patching nonexistent members, patching invalid parameters, etc. All cases are tested (I think). This resolves 833132 and 833376.
-- 
https://code.launchpad.net/~postfuturist/mailman/rest-api-delivery_mode/+merge/72813
Your team Mailman Coders is requested to review the proposed merge of lp:~postfuturist/mailman/rest-api-delivery_mode into lp:mailman.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 15651 bytes
Desc: not available
URL: <http://mail.python.org/pipermail/mailman-coders/attachments/20110825/87555710/attachment.diff>


More information about the Mailman-coders mailing list