Fix bug when current page is empty string

[johngodley]

If you change the current page to an empty string the API request is NaN and the page is cleared.

[Copilot]

Pull request overview

Fixes pagination/state handling so clearing the current-page input doesn’t produce a NaN page request and inadvertently clear table data, by keeping prior query data during refetches and tightening loading/complete status derivation.

Changes:

• Keep previous TanStack Query results during parameter changes via placeholderData in list queries.
• Adjust page-level loading/complete status logic to avoid showing “loading” when placeholder (previous) data exists.
• Make pagination input store a string and commit/clamp the page number on blur to avoid NaN propagation.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/page/redirects/index.tsx	Updates loading/complete status logic to account for placeholder (previous) data.
src/page/logs404/index.tsx	Updates loading status logic to avoid loading state when placeholder data exists.
src/page/logs/index.tsx	Updates loading/complete status logic to account for placeholder (previous) data.
src/page/groups/index.tsx	Updates loading status logic to avoid loading state when placeholder data exists.
src/lib/api/hooks/use-redirect-list.ts	Keeps previous query data during refetch; adds user-facing error reporting.
src/lib/api/hooks/use-logs.ts	Keeps previous query data during refetch for logs and 404s.
src/lib/api/hooks/use-group-list.ts	Keeps previous query data during refetch; adds user-facing error reporting.
src/component/table/navigation/pagination-links.tsx	Prevents empty page input from producing NaN by committing a validated page number on blur.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.