Closed
Bug 543638
Opened 11 years ago
Closed 11 years ago
Thank-you notes for contributions
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect, P3)
addons.mozilla.org Graveyard
Public Pages
Tracking
(Not tracked)
RESOLVED
FIXED
5.8
People
(Reporter: osunick, Assigned: smccammon)
References
Details
Attachments
(2 files, 1 obsolete file)
9.67 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
607 bytes,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
Spec here: http://docs.google.com/Doc?docid=0Ad7mAOXgEBZyZGRzNnZ3YjRfMzBkYnA1Nm5jeA&hl=en TODO: legal review of feature.
Reporter | ||
Comment 1•11 years ago
|
||
DONE: legal review is done
Updated•11 years ago
|
Assignee: nobody → smccammon
Priority: P2 → P3
Assignee | ||
Comment 2•11 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•11 years ago
|
Flags: in-litmus+
Reporter | ||
Comment 3•11 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•11 years ago
|
||
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 5•11 years ago
|
||
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•11 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?
Comment 7•11 years ago
|
||
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•11 years ago
|
||
Now with foreign-key goodness. (and for the record)
Attachment #430114 -
Attachment is obsolete: true
Attachment #433001 -
Flags: review?(clouserw)
Updated•11 years ago
|
Attachment #433001 -
Flags: review?(clouserw) → review+
Comment 9•11 years ago
|
||
Comment on attachment 433001 [details] [diff] [review] Front-end v2 r+. please put the migration in zamboni though. thanks.
Assignee | ||
Comment 10•11 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
Assignee | ||
Comment 11•11 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)
Updated•11 years ago
|
Attachment #433068 -
Flags: review?(clouserw) → review+
Assignee | ||
Comment 12•11 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•11 years ago
|
||
r64267 for updated IPN notify URL
Updated•11 years ago
|
Keywords: push-needed
Updated•5 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•