Skip to content

Search monitors across all pages - #100

Merged
tomschlick merged 1 commit into
masterfrom
feature/bc1b1121
Oct 8, 2026
Merged

tomschlick merged 1 commit into
masterfrom
feature/bc1b1121

Conversation

@tomschlick

Copy link
Copy Markdown
Member

Summary

  • Apply monitors list search at the table level so it queries the full dataset before pagination
  • Cover name, target, description, and tags, with a clearer “Search all monitors” placeholder
  • Add a regression test that finds monitors on other pages (and from page 2 back to page 1)

Test plan

  • Open /admin/monitors with enough monitors to paginate
  • Search for a monitor that only appears on page 2+ and confirm it shows up
  • Search by target, description, and tag as well as name
  • Confirm tests/Feature/Filament/MonitorResourceTest.php passes

@tomschlick
tomschlick enabled auto-merge (squash) October 8, 2026 18:24
@tomschlick
tomschlick merged commit 5cb0b92 into master Oct 8, 2026
2 checks passed
@tomschlick
tomschlick deleted the feature/bc1b1121 branch October 8, 2026 18:25
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.

1 participant