Skip to content

Handle invalid module review page values#15

Closed
phucnguyen1707 wants to merge 1 commit into
profullstack:masterfrom
phucnguyen1707:fix-review-page-pagination
Closed

Handle invalid module review page values#15
phucnguyen1707 wants to merge 1 commit into
profullstack:masterfrom
phucnguyen1707:fix-review-page-pagination

Conversation

@phucnguyen1707

Copy link
Copy Markdown

Summary

  • fall back to page 1 when module review pagination receives a non-numeric page query
  • add regression coverage that ensures Supabase range receives 0..19 for invalid page values

Fixes #14

Tests

  • pnpm --filter @profullstack/threatcrush-web test -- apps/web/src/app/api/modules/[slug]/review/tests/route.test.ts
  • pre-commit hook: Build CLI, Next web build

@ralyodio ralyodio closed this Jun 16, 2026
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.

Module reviews API passes NaN pagination values to Supabase

2 participants