Port to 6.x
cinemasaville - May 23, 2008 - 00:11
| Project: | Simplenews on register |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Description
Hi,
Any possibility of getting this on 6? It seems like a great add-on to simplenews.
Thanks.
#1
D6++
#2
subscribing
#3
I took a first stab at this with the help of the Deadwood module. It definitely needs more testing and is my first attempt contributing back to Drupal so it's probably not 100% up to par.
There is at least one issue, the "User registration page options" section located on /admin/content/newsletters/settings isn't getting themed properly. I'm still looking at it but could use an extra set of eyes (the check boxes should all be in a table with a header/column names above them). I temporarily added titles to the checkbox(es) so that it's useable in the meantime (but will be harder to use if you have multiple newsletters).
Please let me know if there are any issues with the patch. I'm also open for any feedback/help on the theming issue noted above.
Thank you,
Jeff