CVS edit link for afestein

I have written a simple module which adds a Simplenews newsletter subscribe checkbox to the Ubercart checkout process. I think that others would find this useful so I thought I would try to make it available to the public.

The module uses an combination of hook_checkout_pane() and simplenews_subscribe_user() to achieve the objective.

Others have requested this feature here:

http://www.ubercart.org/forum/support/11463/newsletter_subscribe_option_...
http://www.ubercart.org/forum/support/8797/add_newsletter_when_user_regi...

Regards, Adrian

CommentFileSizeAuthor
#1 simplenews_ubercart.zip2.22 KBAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

FileSize
2.22 KB
apaderno’s picture

Status: Postponed (maintainer needs more info) » Needs review

Remember to change the status to needs review, or who reviews the code (who are more than one person) would not know there is code to review.

apaderno’s picture

Status: Needs review » Closed (won't fix)

As for a CVS application, you should use a module that is not simply made of two functions.
Modules like this, which accomplish a single task with few code, are not accepted in CVS repository.

Stevo_0’s picture

Ive installed this, but it seems, after reviewing the code, and testing it out.. like you said "adds a Simplenews newsletter subscribe checkbox to the Ubercart checkout process"..

But doesnt actually update the new user account with the newsletter(s) subscribed to...

No data is actually captured or updated... is that for us to work out?

I have 2 newsletters, and i need them both to be subscribed from ubercart checkout... which automatically adds an account and logs them in, and hopefully subscribe them to newsletters...?

--

Id be happy to pay to have this feature developed, or chip in on someones amazon wishlist. which i see some developers seem to prefer??