Skip to content

perf(issues): cache bounty list response with 30s revalidation window (#716) - #818

Open
PINYOPATTANAWASANPORN wants to merge 1 commit into
MergeFi:mainfrom
PINYOPATTANAWASANPORN:fix/bounty-board-caching-716
Open

PINYOPATTANAWASANPORN wants to merge 1 commit into
MergeFi:mainfrom
PINYOPATTANAWASANPORN:fix/bounty-board-caching-716

Conversation

@PINYOPATTANAWASANPORN

Copy link
Copy Markdown

Summary of Changes

Sets route segment config export const revalidate = 30; on the /issues page (src/app/issues/page.tsx).

Root Cause & Technical Context

IssuesPage fetches the entire backend bounties table via fetchBounties. Because the backend endpoint is currently unpaginated, every page visit and pagination click transfers and parses the entire table on each request. As outlined in #716, until upstream pagination is completed in the backend, adding a 30-second revalidation cache bounds unnecessary round trips and prevents linear server cost growth while keeping bounty statuses and claim updates fresh.

Verification

  • Route segment configuration revalidate = 30 conforms to Next.js App Router caching specifications.
  • Preserves all query filtering, search parameters, mock fallback behavior, and pagination logic.

Closes #716

@vercel

vercel Bot commented Oct 23, 2026

Copy link
Copy Markdown

@PINYOPATTANAWASANPORN is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The bounty board downloads the entire bounties table on every page view and paginates in memory

1 participant