Skip to content

Clean up repo table pagination display#67

Merged
alessandro-ciffo merged 2 commits into
commitvc:mainfrom
wazeerc:chore/repo-table-pagination-cleanup
May 13, 2026
Merged

Clean up repo table pagination display#67
alessandro-ciffo merged 2 commits into
commitvc:mainfrom
wazeerc:chore/repo-table-pagination-cleanup

Conversation

@wazeerc
Copy link
Copy Markdown
Contributor

@wazeerc wazeerc commented May 2, 2026

What changed

  • Extracts pagination state pageIndex, pageSize, pageCount into local variables for cleaner rendering logic.
  • Removes hardcoded magic number 10 and improves readability.

Checklist

  • Tested locally (cd frontend && npm run dev)
  • npm run build passes (for frontend changes)
  • python -m pytest passes (for methodology changes)

### If this changes the methodology:

- [ ] Discussed in an Issue or Discussion first
- [ ] Updated frontend/src/app/methodology/page.tsx to match
- [ ] Updated methodology tests and fixtures

@vercel
Copy link
Copy Markdown

vercel Bot commented May 2, 2026

@wazeerc is attempting to deploy a commit to the Red River West Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
oss-index-supabase-commit Ready Ready Preview, Comment May 13, 2026 8:09am

Request Review

Copy link
Copy Markdown
Contributor

@alessandro-ciffo alessandro-ciffo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the cleanup, @wazeerc!

(The earlier Security failure was unrelated. New Next.js + fast-uri advisories landed right before your run. Fixed on main and now passing here)

@alessandro-ciffo alessandro-ciffo merged commit e435e14 into commitvc:main May 13, 2026
9 checks passed
@wazeerc wazeerc deleted the chore/repo-table-pagination-cleanup branch May 14, 2026 19:46
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.

2 participants