Jump-link icons on clickable stat boxes + make revenue KPIs drillable - #2100
Draft
maebeale wants to merge 3 commits into
Draft
Jump-link icons on clickable stat boxes + make revenue KPIs drillable#2100maebeale wants to merge 3 commits into
maebeale wants to merge 3 commits into
Conversation
…earch The revenue report's KPI tiles need to drill into a filtered registrant list, but the cross-event index ignored payment/scholarship/funder params. Funder keys on scholarships.grant_id: awbw = org-subsidized (no grant), donor = grant-funded. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Clickable stat boxes now show a jump-link icon so it's clear they navigate: always-on for the grid cards (participation, revenue, analytics, ahoy) and hover-revealed on the crowded event stat bar. Revenue KPIs (fees, scholarships, outstanding, org subsidy) now drill into the matching filtered registrant list. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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 registration query filters (payment/funder scopes) + revenue KPIs now link to filtered lists, across 5 stat-box surfaces
Makes clickable stat boxes signal that they navigate (jump-link icon), and makes the revenue report's KPI tiles drillable into a filtered registrant list.
Why
Backend
EventRegistration: newwith_funded_scholarship/with_unfunded_scholarship+ afunderscope (awbw= org-subsidized, no grant;donor= grant-funded). Keyed onscholarships.grant_id.search_by_params(cross-event index) now honorspayment_status,scholarship,funder; addedfunderto the per-event roster too (it already took the first two).Jump icons (all 5 clickable stat-box types) —
fa-arrow-up-right-from-square, top-right, matching the dashboard convention:Revenue KPIs now link (year-level → cross-event index, carrying
event_idwhen filtered to one event):payment_status=paid· Outstanding →payment_status=unpaid· Scholarships →funder=donor· Org subsidy →funder=awbw.Note: a dollar figure vs. a registrant count isn't a 1:1 reconciliation (a partly-paid registrant adds to "Fees collected" but shows under
payment_status=unpaid); the org-subsidy link lists only the unfunded-scholarship recipients, not the discount portion. Same approximation the participation KPIs already make.