Hi David,<br><br><div>Thanks for the detailed review!</div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
- The configuration UI works okay, but it would be nice if all the options relating to commenting were available in one place (including, e.g., things like which content types have comments enabled, whether or not moderation is enabled, etc.)  We can always write a form adapter that saves these options however we like, even if the settings are also available in the other places they are now, like the Types configlet.<br>


<br></blockquote><div><br></div><div>We spoke about this when Timo built this part. My advice then was that as long as p.a.d was a separate package, duplicating what was already in the Types control panel would be confusing. </div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">- Currently moderation is a global setting (based on whether the Comment type has the comment_moderation_workflow or not).  It would be nice to be able to configure it so that comments are moderated for anonymous users, but not for users with some particular permission (e.g. &quot;plone.app.discussion: Skip moderation&quot;).  For users with that permission, comments with a moderation workflow would get automatically published when the comment is submitted.<br>

</blockquote><div><br></div><div>This can be done with an automatic transition guarded by this permission.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

- Is there a recommended way to provide e-mail notifications based on commenting (e.g. e-mail to the site owner when a new comment is submitted for moderation; e-mail to the commenter when it is approved; e-mail to the commenter when someone posts a reply...)</blockquote>

<div><br></div><div>I hope we can make that happen with content rules. We may need to register some new event types for the content rules engine, but that&#39;s easy enough.</div><div><br></div><div>Martin</div></div></div>