Add pagination to server list

[Boy132]

Closes #1954

Was easier than expected :D

[coderabbitai]

📝 Walkthrough

Walkthrough

The server list page now enables pagination with layout-dependent configuration. Previously pagination was disabled entirely. Grid layouts display options [10, 20, 30, 40] defaulting to 10; other layouts display [10, 20, 50, 100] defaulting to 20.

Changes

Cohort / File(s)	Summary
Server List Pagination app/Filament/App/Resources/Servers/Pages/ListServers.php	Modified table() method to configure conditional pagination based on dashboard layout instead of disabling pagination; grid layout receives [10, 20, 30, 40] with default 10, while other layouts receive [10, 20, 50, 100] with default 20

Pre-merge checks

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Add pagination to server list' directly and specifically describes the main change in the pull request: implementing pagination for the server list view.
Description check	✅ Passed	The description references the linked issue (#1954) which outlines the pagination implementation objective and provides meaningful context about the work.
Linked Issues check	✅ Passed	The code changes implement pagination for the server list by configuring paginated() with layout-specific options and defaults, directly fulfilling the objective stated in issue #1954.
Out of Scope Changes check	✅ Passed	All changes are scoped to pagination configuration in the ListServers table() method; no unrelated modifications detected outside the stated objective.

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6c02f9a and 950c8b9.

📒 Files selected for processing (1)

• app/Filament/App/Resources/Servers/Pages/ListServers.php (1 hunks)

🔇 Additional comments (1)

app/Filament/App/Resources/Servers/Pages/ListServers.php (1)

120-123: Pagination configuration aligns with Filament’s API and looks solid.

->paginated($usingGrid ? [10, 20, 30, 40] : [10, 20, 50, 100]) together with ->defaultPaginationPageOption($usingGrid ? 10 : 20) matches the documented pattern for customizing per-page options and default page size, and your defaults (10 / 20) are correctly included in the respective option sets. (docs.laravel-filament.cn) The layout‑dependent choices (smaller pages for grid, larger for list) also make sense given the different densities.

Please just confirm this still lines up with the exact Filament version you have locked in your composer.lock, in case of any future API changes.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[notAreYouScared]

👍