Freegle / Iznik / 5071

70%

Build Type

push

circleci

Committed by invalid-email-address

Commit Message

fix(donations): distinguish PayPal/Stripe/external in thank-you email

V1 had three separate "please thank" emails worded according to the
donation source:

  - donations.php PUT     → "via an external donation"
  - donateipn.php (PayPal)→ "via PayPal Donate"
  - stripeipn.php         → "via Stripe"

V2 had collapsed all three into a single email_donate_external task
using DonateExternalMail, which hardcoded "via an external donation" —
so real PayPal and Stripe donors were being mislabelled as external.

- Go: PayPal IPN, Stripe IPN, and manual external all now pass a
  source field ('paypal' / 'stripe' / 'external') on the queued task.
- Laravel: DonateExternalMail reads source (defaults to external for
  backwards compat on older queued tasks) and renders the appropriate
  channel phrase in subject and MJML body.
- Tests: Vitest-side expectations for all three sources in
  ProcessBackgroundTasksCommandTest; Go-side assertions added in
  paypalipn_test, stripeipn_test, and donations_test confirming the
  queued task carries the right source value.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Coverage Stats

13290 of 20278 branches covered (65.54%)

Branch coverage included in aggregate %.

12 of 13 new or added lines in 5 files covered. (92.31%)

11 existing lines in 3 files now uncovered.

95792 of 132187 relevant lines covered (72.47%)

21.81 hits per line