Posted by jchatard on April 14, 2009 at 1:29pm
Jump to:
| Project: | MailChimp |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Hi all,
Here is a small patch that maps Dropdown et Radios from MailChimp to Drupal FormAPI. With that newsletter subscription form displays nice widgets.
i.e. : no more textfield input when telling MailChimp form builder to render a dropdown.
I also added a test condition so that hidden fields (from MailChimp) renders hidden fields in Drupal.
Unfortunaltely, the MailChimp date input can't be supported easily for now.
Last, I just made a small modification to form validate, which was letting me subscribe to a newsletter even if I did not checked the box (the small '1' added).
Look at the patch.
Thank you guys!
Jérémy
| Attachment | Size |
|---|---|
| mailchimp-Drupal-FormAPI-6.diff | 3.05 KB |
Comments
#1
grr, patch just takes a 'C'.
#2
Nice patch, applied. Needs some review. Also might want to add an switch for dates. Note applied to 6.x-2.x-dev
#3
#4
Automatically closed -- issue fixed for 2 weeks with no activity.