Skip to content

Automated scholarship agreement emails - #2193

Draft
maebeale wants to merge 1 commit into
maebeale/scholarship-decline-emailfrom
maebeale/scholarship-agreement-emails
Draft

Automated scholarship agreement emails#2193
maebeale wants to merge 1 commit into
maebeale/scholarship-decline-emailfrom
maebeale/scholarship-agreement-emails

Conversation

@maebeale

Copy link
Copy Markdown
Collaborator

🤖 suggested review level: 3 Read 📖 additive notification/mailer wiring on top of the decline PR

Stacked on #2178 (base = maebeale/scholarship-decline-email). Merge that first.

What

The automated emails split out of #2178:

  • Decline → admin FYI (scholarship_agreement_declined_fyi) with the reason.
  • Sign → recipient confirmation (with a link back to their ticket) + admin FYI, sent once on the first sign.
  • Contact-us threading: the decline page's "contact us" link carries the registration through to the contact form and the contact FYI email, so the team lands on the right registration.

How

  • New Notification kinds + NotificationMailer methods + HTML/text views + previews, wired through NotificationMailerJob; EMAIL_TOPICS entries for the admin filter.
  • Controller wiring in events/callouts#sign_agreement / #decline_agreement.
  • Contact-us hidden field + mailer lookup + FYI link.

No schema changes — purely additive code on top of the decline feature.

Stacked on the decline PR. On decline, email the admin team an FYI. On
sign, confirm to the recipient (with a link back to their ticket) and send
the team an FYI. Thread the decline page's "contact us" link through to the
contact form + FYI so the team lands on the right registration.

Adds the notification kinds, mailers + views + previews, the controller
wiring, and the contact-us registration threading.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant