18 votes · 18 up · 0 down
Maintenance Override: Immediate Merge
This PR fixes a critical infrastructure bug, not content.
The pagination limit was hiding 10+ PRs from the voting dashboard (including #13 and #47). Users couldn't vote on what they couldn't see. That's not chaos - that's broken.
Policy: Content PRs = Sunday votes only. Infrastructure fixes that restore fair voting = immediate merge.
Credit to @Kl0ven who spotted this issue first on Day 4.
🤖 OpenChaos Bot
Summary: The PR modifies the getOpenPRs function to paginate through the GitHub API to retrieve all open pull requests, as the API returns a limited number of PRs per request. It also adds better error handling for the GitHub API.
Files changed: 1 (src/lib/github.ts)
Impact: Medium - This affects the core functionality of fetching open PRs, which is essential for the app.
Maintenance Override: Immediate Merge
This PR fixes a critical infrastructure bug, not content.
The pagination limit was hiding 10+ PRs from the voting dashboard (including #13 and #47). Users couldn't vote on what they couldn't see. That's not chaos - that's broken.
Policy: Content PRs = Sunday votes only. Infrastructure fixes that restore fair voting = immediate merge.
Credit to @Kl0ven who spotted this issue first on Day 4.
🤖 OpenChaos Bot
Summary: The PR modifies the getOpenPRs function to paginate through the GitHub API to retrieve all open pull requests, as the API returns a limited number of PRs per request. It also adds better error handling for the GitHub API.
Files changed: 1 (src/lib/github.ts)
Impact: Medium - This affects the core functionality of fetching open PRs, which is essential for the app.
Fix pagination bug hiding older PRs
GitHub API defaults to 30 results per page. With 37+ open PRs, older ones like #13 were being hidden. This implements proper pagination to fetch all PRs.
getOpenPRs() in src/lib/github.ts to loop through all pagesper_page=100 and increments page number until no more resultsgetPRVotes()