[Merge] lp:~flo-fuchs/mailman.client/settings into lp:mailman.client
Barry Warsaw
barry at canonical.com
Tue May 31 20:19:23 CEST 2011
Review: Approve
This looks great; a couple of comments:
* See above for recommendations against u'' strings and for using print instead of returning strings directly in doctests.
* Could you please open a bug (tagged with 'mailman3') on the need to expose an API call to look up a member-id given an email address? I'll make sure to expose that in a8.
With the above consideration, I'll approve this change. Thanks! And feel free to land it.
--
https://code.launchpad.net/~flo-fuchs/mailman.client/settings/+merge/56088
Your team Mailman Coders is subscribed to branch lp:mailman.client.
More information about the Mailman-coders
mailing list