Skip to contentSkip to conversation

PATCH: Notify moderation

PATCH Ben Schmidt 2 participants 3 messages Jan 26, 2010 – Mar 9, 2010

lore
Share & tools
lore
#0 lore raw

This patch allows posters to be sent notification when their posts are held for moderation. Requires new listtext.

Tests fine for me (as have all the patches I have sent, though I neglected to mention once or twice).

Could this be included in mlmmj?

I realise I'm sending quite a few patches for consideration here! I believe these are all useful additions, though. They are features that are present in other mailing list programs, and which I need and use.

This will be all for a little while, too. The few other things I'm interested in are lower priority. I'm keen to have these patches I've already sent included in mlmmj (after discussion and modification as appropriate)!

Ben.

TUNABLES +5 -0
diff -r 2918436fa450 TUNABLES
@@ -103,6 +103,11 @@
    If this file is present, the owner(s) will get a mail with the address of
    someone sub/unsubscribing to a mailinglist.
 
+ ��� notifymod			(boolean)
+
+   If this file is present, the poster (based on the envelope from) will
+   get a mail when their post is being moderation.
+
  ��� digestinterval		(normal)
 
    This file specifies how many seconds will pass before the next digest is
contrib/web/perl-admin/conf/tunables.pl +5 -0
diff -r 2918436fa450 contrib/web/perl-admin/conf/tunables.pl
@@ -100,6 +100,11 @@
 			  "Notify subscribers",
 			  "If this option is set, the owner(s) will get a mail with the address of someone sub/unsubscribing to a mailinglist.");
 
+mlmmj_boolean("notifymod",
+			  "Notify moderation",
+			  "If this option is set, the poster (based on the envelope from) will ".
+			  "get a mail when their post is being moderation.");
+
 mlmmj_string("digestinterval",
 			 "Digest interval",
 			 "This option specifies how many seconds will pass before the ".
contrib/web/php-admin/conf/tunables.pl +5 -0
diff -r 2918436fa450 contrib/web/php-admin/conf/tunables.pl
@@ -100,6 +100,11 @@
 			  "Notify subscribers",
 			  "If this option is set, the owner(s) will get a mail with the address of someone sub/unsubscribing to a mailinglist.");
 
+mlmmj_boolean("notifymod",
+			  "Notify moderation",
+			  "If this option is set, the poster (based on the envelope from) will ".
+			  "get a mail when their post is being moderation.");
+
 mlmmj_string("digestinterval",
 			 "Digest interval",
 			 "This option specifies how many seconds will pass before the ".
listtexts/en/moderation-poster new file +11 -0
diff -r 2918436fa450 listtexts/en/moderation-poster
@@ -0,0 +1,11 @@
+Subject: Post waiting for approval
+
+Hi, this is the mlmmj program managing the mailinglist
+
+$listaddr$
+
+This list is configured to have moderated posts, the post has been queued
+for approval.
+
+--- Below this line are the first 100 lines of the message up for moderation --->
+$originalmail$
src/mlmmj-process.c +52 -16

84 lines not on this page · raw message

#1 lore raw
Quoted 5 lines · Ben Schmidt

This patch allows posters to be sent notification when their posts are held for moderation. Requires new listtext.

Tests fine for me (as have all the patches I have sent, though I neglected to mention once or twice).

Could this be included in mlmmj?

Of course. As soon as the BMFH (Maintainer, not Operator :) have reviewed them and ACK'ed them, they'll be included and you'll get due credit in the changelog :)

Quoted 3 lines

I realise I'm sending quite a few patches for consideration here! I believe these are all useful additions, though. They are features that are present in other mailing list programs, and which I need and use.

This will be all for a little while, too. The few other things I'm interested in are lower priority. I'm keen to have these patches I've already sent included in mlmmj (after discussion and modification as appropriate)!

Thanks for all the work so far--it's really great with the contributions. Looks like we'll see mlmmj-1.2.18-RC1 as soon as Morten gets around to it :)

Signature · 4 lines
Mads Martin Joergensen, http://mmj.dk
"Why make things difficult, when it is possible to make them cryptic
 and totally illogical, with just a little bit more effort?"
                                 -- A. P. J.
#2 lore raw
In reply to Ben Schmidt

I'd just like to give a gentle bump regarding this patch, particularly as the later richer listtext patches won't apply cleanly without this one.

Can you spare a few minutes to have a look at this and commit it or discuss it, Morten?

Ben.

Quoted 20 lines · Ben Schmidt

This patch allows posters to be sent notification when their posts are held for moderation. Requires new listtext.

Tests fine for me (as have all the patches I have sent, though I neglected to mention once or twice).

Could this be included in mlmmj?

I realise I'm sending quite a few patches for consideration here! I believe these are all useful additions, though. They are features that are present in other mailing list programs, and which I need and use.

This will be all for a little while, too. The few other things I'm interested in are lower priority. I'm keen to have these patches I've already sent included in mlmmj (after discussion and modification as appropriate)!

Ben.