Add CE request/payment deadlines to events by maebeale · Pull Request #1870 · rubyforgood/awbw

maebeale · 2026-06-22T13:57:18Z

🤖 PR, suggested 👤 review level: 🔬 Inspect — reworked onto main's CE cutover (new continuing_education_registrations model); logic + copy changes across card, callout, page, and emails

What is the goal of this PR and why is this important?

Lets admins set two per-event CE deadlines — request CE credit by and CE payment due by — so registrants are nudged before each cutoff.

How did you approach the change?

New date columns ce_hours_request_deadline / ce_payment_due_deadline on events; permitted in EventPolicy, edited in the CE callout section of the event form.
Surfaced across the new (post-cutover) CE surfaces:
- CE card (MagicTicketCallouts): payment deadline appended to the amount-due badge until ce_paid_in_full? ($150 due by Aug 15); request deadline appended to the license-needed badge (License number needed by Jul 1).
- CE callout page: "Request CE credit by …" on the not-yet-requested branch; "Payment due by …" beside the pay button while a balance remains.
- Public CE page (ce_hours): deadline panel.
- Confirmation + reminder emails: shared _ce_deadlines partial, gated on event.ce_eligible? (not the individual registrant — the confirmation fires before anyone opts into CE, so event-level is the correct nudge audience).

Anything else to add?

Framing shift from main's CE cutover: hours are now event-set (ce_hours_offered), not registrant-requested, so "request your hours by" became "request CE credit by" (the outstanding registrant step is now the professional license). Column names kept as-is.
Deadlines only render when set; events without them are unchanged.

maebeale · 2026-06-22T13:57:28Z

    if complete
      return unless amount_cents.positive?
-      return "#{amount} due"
+      return "#{amount} due" if registration.paid_in_full? || event.ce_payment_due_deadline.blank?


🤖 From Claude: The deadline suffix is suppressed once paid_in_full?. CE payment isn't tracked separately from the event fee, so this reuses the same signal callouts/ce already uses to show CE "Paid".

maebeale · 2026-06-22T14:05:27Z

+    shown in the confirmation and reminder emails. Mirrors the CE callout's
+    deadline copy. Locals: event (decorated), registration. The deadlines are
+    plain dates, so no time zone is applied. %>
+<% if registration.ce_credit_requested? && (event.ce_hours_request_deadline || event.ce_payment_due_deadline) %>


🤖 From Claude: Gated on ce_credit_requested? so only CE registrants see deadlines, and on at least one deadline being set so events without them are unchanged. Same partial backs both confirmation and reminder (html + text) to keep the four templates in sync.

maebeale · 2026-06-22T14:05:32Z

+    shown in the confirmation and reminder emails. Mirrors the CE callout's
+    deadline copy. Locals: event (decorated), registration. The deadlines are
+    plain dates, so no time zone is applied. %>
+<% if registration.ce_credit_requested? && (event.ce_hours_request_deadline || event.ce_payment_due_deadline) %>


🤖 From Claude: Gated on ce_credit_requested? so only CE registrants see deadlines, and on at least one deadline being set so events without them are unchanged. One partial backs both confirmation and reminder (html + text) to keep the four templates in sync.

Copilot