Thank-you notes for contributions

RESOLVED FIXED in 5.8

Status

defect
P3
normal
RESOLVED FIXED
9 years ago
3 years ago

People

(Reporter: osunick, Assigned: smccammon)

Tracking

unspecified
Dependency tree / graph
Bug Flags:
in-litmus +

Details

Attachments

(2 attachments, 1 obsolete attachment)

Reporter

Comment 1

9 years ago
DONE: legal review is done
Assignee: nobody → smccammon
Priority: P2 → P3
Assignee

Comment 2

9 years ago
A couple of observations:

First, emailing amo-unsubscribe@moco resulted in a quick reply from Nick,  confirming that we don't have an automatic system in place for handling opt-out requests. I see the manual $email_blacklist in site config, and will just rely on that for now.

Second, from the spec:

"The PayPal IPN contains the email address of the user making the payment- we'll have to use this to generate the thank you email.  We will only retain this data long enough to send the thank-you email and then we will clear it from the database."

This very short-term email retention policy is in conflict with current contributions stats functionality, and with the requirements for the stats redesign in bug 540885. Which is more important? What is a good compromise?

Updated

9 years ago
Flags: in-litmus+
Reporter

Comment 3

9 years ago
I've updated bug 540885, we should not be retaining email info for longer than we need to in order to send the email.  Thank you notes are never retroactively sent so this should be ok.
Assignee

Comment 4

9 years ago
Posted patch Remora front-end (obsolete) — Splinter Review
Since the contributor's locale is useful and interesting beyond just thankyou notes, I added it to the stats table as `source_locale`.

The back-end mailer will be written for Zamboni.
Attachment #430114 - Flags: review?(clouserw)
Comment on attachment 430114 [details] [diff] [review]
Remora front-end

If thankyou_note is a translated field it should be a FK to the translations table.  But the reason for the r-, if I add a locale to the thank-you note box, the whole box turns red, and "null" shows up in red in the bottom right.
Attachment #430114 - Flags: review?(clouserw) → review-
Assignee

Comment 6

9 years ago
(In reply to comment #5)
> But the reason for the r-, if I add a locale to the thank-you note box,
> the whole box turns red, and "null" shows up in red in the bottom right.

I don't get this result at all. Do you get any javascript errors? For any locale or just certain ones?
Apparently I'm the only one with problems here, so r+ after the FK fix.  Let's see what it does on preview and with QA's eyes.
Assignee

Comment 8

9 years ago
Posted patch Front-end v2Splinter Review
Now with foreign-key goodness. (and for the record)
Attachment #430114 - Attachment is obsolete: true
Attachment #433001 - Flags: review?(clouserw)
Attachment #433001 - Flags: review?(clouserw) → review+
Comment on attachment 433001 [details] [diff] [review]
Front-end v2

r+.  please put the migration in zamboni though. thanks.
Assignee

Comment 10

9 years ago
r64257 for the front-end

6062b353 for the zamboni migration
accf6994 and 02bf3f3f for the zamboni back-end thankyou mailer
ef3a8b09 for the zamboni PayPal service

For deployment, IPN will need to be reconfigured (or a rewrite put in place):

/services/paypal.php => /z/services/paypal
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Keywords: push-needed
Resolution: --- → FIXED
Depends on: 552974
Assignee

Comment 11

9 years ago
This patch tells PayPal IPN to notify /services/paypal soon to be handled by zamboni. This triggers thankyou emails.
Attachment #433068 - Flags: review?(clouserw)
Attachment #433068 - Flags: review?(clouserw) → review+
Assignee

Comment 12

9 years ago
QA Notes:

The PayPal service script has been ported to zamboni in this bug. This could affect contribution stats. We will need to test whether real transactions complete and show up in the stats dashboard.

Also, please wait for bug 552974 to be resolved before testing with real PayPal transactions. The developer front-end can be tested now, however.
Assignee

Comment 13

9 years ago
r64267 for updated IPN notify URL
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.