Rework bulk payment ticket around the payer#2008
Merged
Merged
Conversation
The viewer of a bulk payment ticket is the payer covering other people, not an attendee — so the ticket now leads with the event context card, then the payer identity (name, organization, count of registrants covered) and payment status as the headline, followed by the per-person registrant details and the grouped invoice/submission document links. Drops the field labels on the payer block in favor of a prominent name + secondary org line. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
maebeale
marked this pull request as ready for review
July 22, 2026 12:46
- Move the perforation "ticket tear" up so it separates the event card from the payer/payment portion, and drop the divider between the payer and the registrant list (they're one unit — the payer covers these registrants). - Registrant details count now reads "(listed of covered)", e.g. "(3 of 3)", since the covered total is already stated above. - Add an explicit "Admin only" badge to the (already admin-gated) payment allocations box. - Payment badge: paid-but-unallocated now reads "unallocated" (not "remaining") and is orange to signal it needs action. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…/time - Replace the "View submission" card with a plain "View your form responses" text link grouped with the resend-confirmation action, matching the registration ticket. Leaves the invoice as the sole document card. - Event card date/time now reads on one row as "<date range> (<time tz>)" via date_range + times, dropping the styled two-row layout whose oversized timezone abbreviation looked off. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Add EventDecorator#short_date_range (same collapsed range as date_range but without the weekday prefix) and use it for the compact event card, where the weekday and inline time were noise. 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: 3 Read 📖 view-only restructure of the bulk-payment ticket + a shared payment badge (text/color); no logic/data changes
What
Reworks the public bulk-payment ticket (
events/bulk_payments/ticket.html.erb) around the fact that the viewer is the payer covering other people, not an attendee.(listed of covered)e.g.(3 of 3)._payment_badge: the paid-but-unallocated state now reads "unallocated" (not "remaining") and is orange to signal it needs action. Also affects the bulk payments dashboard card.Why
Unlike a registration ticket, the person reading this is paying on behalf of others, so payer identity and what they owe matter more than event details.
Notes