Conversation
…gment checkbox Adds a photo-waiver path to the host reimbursement submit flow. When the required event photos (group/box_stack/pizza + the >=5 additional) are missing, the host can tick an acknowledgment checkbox to submit anyway; receipt, attestation, payment-method and attendance gates stay required. Backend: new nullable `photos_waived_at` on payouts (migration applied to prod), stamped at submit/create when `photosWaived` is set and cleared on unsubmit/photo-complete submit. `getReimbursementReadiness` gains `readyToSubmitWithoutPhotos`; both submit endpoints gate on it when waived. Frontend: PayoutsTab shows the waiver Checkbox when photos are missing and enables submit; admin /payments shows a "Submitted without photos" pill via SubmittedForReviewBadge on the city row, co-host chip, ledger row and review modal. i18n keys added across all 8 locales. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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.
Summary
Lets a host submit their reimbursement for payment on /payments even when the required event photos are missing — provided they tick an acknowledgment checkbox confirming they understand they're submitting without the required photos.
Receipt, attestation, payment-method and attendance gates are unchanged — only the photo gates (group / box_stack / pizza role photos and the ≥5 additional event photos, tiramisu-58530) become waivable.
Behavior
Checkboxappears in the submit card. Ticking it enables Submit for review. The missing-requirements list stays visible.payouts.photos_waived_atis stamped at submit/create when waived, and cleared on unsubmit or a photo-complete submit. The amberSubmittedForReviewBadgereads "Submitted without photos" on the by-city row, co-host chip, ledger row and review modal so reviewers know photos are missing.Backend
schema.prisma:Payout.photosWaivedAt(photos_waived_attimestamptz, nullable).getReimbursementReadinessgainsreadyToSubmitWithoutPhotos(everything except photos);readyToSubmitkeeps its full meaning.POST /reimbursement/submit+ legacyPOST /payouts: acceptphotosWaived; when true, skip the photo gates and gate on the non-photo requirements; stamp/clearphotosWaivedAt.POST /reimbursement/unsubmitclears it.serializePayouts emitphotosWaivedAt.Frontend
Payout.photosWaivedAttype (inherited byAdminPayout);ReimbursementReadiness.readyToSubmitWithoutPhotos;submitReimbursement(partyId, attested, photosWaived?).SubmittedForReviewBadgephotosWaivedAtprop + 4 call sites wired.payouts.photoWaiverAck/payouts.submitNeedsPhotoWaiverin all 8 locales.DB / deploy
payouts.photos_waived_at, nullable timestamptz) — verified. Safe to merge; backend auto-deploys from master.Verify on preview
Plan:
plans/cannelloni-58543-photo-waiver.md🤖 Generated with Claude Code