Add scholarship decline flow with admin email and totals exclusion - #2178
Draft
maebeale wants to merge 2 commits into
Draft
Add scholarship decline flow with admin email and totals exclusion#2178maebeale wants to merge 2 commits into
maebeale wants to merge 2 commits into
Conversation
Recipients can decline a scholarship agreement with a reason from their scholarship page. Declining emails the admin team an FYI, records the reason, zeroes the allocation so the award drops out of every total, and shows a Declined badge everywhere scholarships appear. Editing the award amount re-offers it and clears the decline. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Editing the award amount re-offers the scholarship and discards the recipient's recorded decline; warn the admin before that happens. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 suggested review level: 5 Inspect 🔬 new decline state that alters financial totals across dashboards, grant budgets, revenue/scholarship reports, and many display surfaces
What is the goal of this PR and why is this important?
How did you approach the change?
decline_agreementaction + route; native<details>disclosure reveals a required reason box next to Agree.Scholarship#decline_agreement!stampsagreement_declined_at/reason, clears any signed state, and zeroes the allocation so all allocation-based totals self-correct.scholarship_agreement_declined_fyiNotification kind →NotificationMailermethod + HTML/text views + preview, wired throughNotificationMailerJob..not_declinedadded to theamount_cents/scope readers — grant budget/remaining, funding split scopes, EventDashboard, EventScholarshipFigures, EventRevenueFigures, ScholarshipsGrouping, grant decorator counts, registrant award badge.ScholarshipDecorator.scholarship-decline-guardStimulus controller warns the admin before saving that change so the decline isn't discarded silently.Anything else to add?
agreement_declined_at+agreement_declined_reasontoscholarships.