Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi - I used your module as a basis for a custom module for a client of mine. I added some functionality, including the export function. I was wondering if you would like to merge it. You can download it at this link http://www.fightingcrane.com/emtest/drupal-mods/mailing_list_enhanced.ta....
Comment | File | Size | Author |
---|---|---|---|
#17 | mailing_list.nl_.po | 12.63 KB | Robin van Emden |
#16 | mailing_list-302770-16.patch | 15.94 KB | Robin van Emden |
#10 | mailing_list_unsubscribe_notify.patch | 15.49 KB | emichan |
#5 | mailing_list_feature_add.patch | 25.99 KB | emichan |
Comments
Comment #1
litwol CreditAttribution: litwol commentedI would love to use your code! can you please create a patch that i can use ? http://drupal.org/patch
Comment #2
sprevrha CreditAttribution: sprevrha commentedCan you provide info on how to use this? I added a mailing list in 'site configuration' and the admin and owner user rules to allow mailing list administration, I am logged as admin now what do I do??
Cheers and thanks for your support!!
Sven
Comment #3
emichan CreditAttribution: emichan commentedHi - sorry for the slow response. I'll create a patch and upload it as soon as possible. I haven't contributed code before, so thanks for pointing me in the right direction.
Comment #4
emichan CreditAttribution: emichan commentedA block should be automatically created for each mailing list. The block provides a subscribe form - just a textbox for an email address and a submit button. You can add the blocks to any page you want.
Comment #5
emichan CreditAttribution: emichan commentedOkay, I've created a patch based on the latest development snapshot available. Unfortunately, it's just one big monolithic patch for all the changes I made - I hope that doesn't cause any problems. Just let me know if there's anything else I can do to help out.
Comment #6
litwol CreditAttribution: litwol commentedComment #7
litwol CreditAttribution: litwol commentedhello emichan,
I commited your other 2 patches so far. it sounds that some parts of this patch are alread commited (for example export functionality?). Could you review what needs to be updated and reroll this patch? Please coordinate with #531794: Admin UI for individual emails + list import feature so you guys dont run into conflicts.
Comment #8
emichan CreditAttribution: emichan commentedHi litwol,
I'm in the process of rerolling this patch against HEAD. You've had a few more commits, so I'm having to redo it again. ;) I'll get it posted shortly.
emichan
Comment #9
litwol CreditAttribution: litwol commentedThank you!. yes now is definitely a good time to reroll as i started doing active review and commit of patches.
Cheers and thanks for your contribution :)
Comment #10
emichan CreditAttribution: emichan commentedI rerolled the patch against the latest head. It patched cleanly on my test machine. The admin ui patch duplicated some of the functionality I added, and export was already included. So, this patch adds an unsubscribe form for each mailing list and adds the option to send notification on successful (un)subscription.
Comment #11
litwol CreditAttribution: litwol commented@emichan it seems your patch didnt upload right. can you re-upload please.
Comment #12
litwol CreditAttribution: litwol commentedHere is the path to the patch uploaded in #10 : http://drupal.org/files/issues/mailing_list_unsubscribe_notify.patch
Comment #13
Jaza CreditAttribution: Jaza commented@emichan
(pasted from my comments to @litwol on IRC):
I'm not sure that all the 'notify on (un)subscribe' stuff is really a necessary addition to the module. A better approach for anyone wanting notify functionality, would be to tie in mailing_list with actions... this would probably only require inventing some hook_mailing_list_*() events, and then implementing them in a separate module. It would also have the massive advantage of allowing any action to occur on subscribe / unsubscribe, rather than limiting it to just 'send an e-mail to 1 person notifying them'.
So emichan, my advice for you is to modify your patch so that:
I also think that the unsubscribe stuff should be moved into a separate issue, and that it should wait until we have a solid verification system in mailing_list. But that's just my thoughts. :P
Comment #14
litwol CreditAttribution: litwol commentedI'm fine with subdir, assuming we chose to go that way.
Comment #15
emichan CreditAttribution: emichan commentedHey,
The suggestions for notification makes sense to me. I had it in my original modifications to the module and just thought I'd keep it in there.
As for the unsubscribe link. I actually think it is fairly important to have. If you think it's better to implement it in a different way, then that's okay. But there needs to be an easy way to allow people to unsubscribe themselves from a mailing list - even more because there is not a verification system in the module. At least in the US, the CAN-SPAM act requires that there is an "unsubscribe mechanism" in place. I'm open to ideas for a better way to handle that.
I'll make the changes to the notification aspect and resubmit the patch - fair warning it may be a few days before I can get to it. :)
Comment #16
Robin van Emden CreditAttribution: Robin van Emden commentedHi! I applied the #12 patch, but due to a permissions problem nobody was allowed to open the unsubscribe page.
Changing one line and adding another made everything work though:
return array('administer mailing lists','unsubscribe from mailing lists');
and added:
.'access arguments' => array('unsubscribe from mailing lists'),
to the mailing-list/%mailing_list/unsubscribe menu item.
Attached: rerolled patch to head (August 5) including these small changes.
Thanks a lot both for the module and for the #12 patch!
Comment #17
Robin van Emden CreditAttribution: Robin van Emden commentedAttached: a Dutch translation of the mailing_list module, including translations of the added functionality in the #12 / #16 patches.
Comment #18
lwiegand CreditAttribution: lwiegand commentedComment #19
iMiksuRobin van Emden, thanks for your effort! I was just about to code that funcitonality myself but thank god you code that already ;)
I hope this patch get ported to HEAD.
Comment #20
Anonymous (not verified) CreditAttribution: Anonymous commentedHi all
great module that i use :-)
I notice that notifications are really important :-)
is it possible for D7?
Best regards
ArchGalileu
www.gasparsantos.eu
www.quartetodouro.eu
www.cameratanovnorte.eu
Comment #21
manuel.adanClosing this as outdated, 6.x version is no longer active.