Reduce duplicate API requests across event management pages - #1314
Merged
Conversation
Adds a lightweight event counts endpoint for the sidebar badges, caches the event and organizer queries properly, and adds a superadmin account verification toggle.
|
π Deployed to the Hi.Events-pr-1314 environment in Hi.Events
1 service not affected by this PR
|
railway-app
Bot
temporarily deployed
to
Hi.Events / Hi.Events-pr-1314
August 13, 2026 06:10
Destroyed
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Performance
useGetEventhadstaleTime: 5(five milliseconds), so/api/events/{id}refetched on every route change and twice on first load. Removed, now uses the 60s default.GET_EVENT_QUERY_KEYinvalidation to the 15 product, category, image and occurrence mutations, since the event payload embeds those relations and nothing refreshed them before.useGetOrganizerno longer fires/api/organizers/undefined(500) before the event loads, and no longer setsstaleTime: 0, gcTime: 0.Event counts endpoint
GET /events/{event_id}/countsreturning lifetime order and attendee totals fromevent_statistics.Admin
PUT /admin/accounts/{account_id}/verification(superadmin only).is_manually_verifiedcould only be set by Stripe sync.Tests
saasOnly()alongsidenonSaasOnly().Check-in lists