[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