Comment 5 for bug 333378

I agree, that's better.

--Paul

On Feb 24, 2009, at 9:57 AM, Chris Rossi wrote:

> Thinking about the user story for this, I know that it would be easier
> for me to grok what had occurred if the email presented not just the
> before and after states but a summary of what had changed:
>
> The moderators for the Paul Private community have changed:
>
> Fred is no longer a moderator.
> Wilma is no longer a moderator.
> Barney is now a moderator.
>
> Previous Moderators:
> Fred
> Linus
> Lucy
> Wilma
>
> New Moderators:
> Barney
> Linus
> Lucy
>
> I will implement such if there is no objection.
>
> --
> Email for moderator changes
> https://bugs.launchpad.net/bugs/333378
> You received this bug notification because you are a direct subscriber
> of the bug.
>
> Status in Porting KARL to a new architecture: New
>
> Bug description:
> Send an email whenever the moderator information changes.
>
> Specification
> ===============
>
> - Sent when there is a change of moderation status (add a moderator,
> remove a moderator) on any of the moderators
>
> - Email is To: all new and old moderators. Meaning, if someone had
> their moderation status removed, they also get the email
>
> - The email should look like this:
>
> - The To: line has all the old/new moderators, full name plus email
>
> - The From: line is the community email address, community-
> id@system_email_domain
>
> - Subject is "Change in moderators for Community Title"
>
> - The body shows: a link to the community, a listing of the
> previous list of
> moderators (with links to profile), and ditto for the new list
>
> - HTML-only, no plaintext version
>
> - There is an attachment to this issue that shows the HTML formatting
>
>
> Implementation notes for ChrisR
> ================================
>
> - Free to refactor
>
> - Code will go in src/karl/karl/views/members.py, in def
> manage_members_view
>
> - You get to this in the browser by going to /communities/default,
> click on "Members" link, then click on "Manage Members"
>
> - Look for:
>
> for action in c_moderators:
> if action['remove']:
>
> - I generated the emails using ZPTs, in src/karl/karl/views/
> templates/email_*.pt
>
> - Find any ways to improve the tests
>
> - I will include an example of what the emails should look like
>
> - Make sure you ask questions via this task, I'll subscribe
>