Search: loading indicator and performance note (#32) #72

Merged
poslop merged 1 commits from issue-32-search-indicator into main 2026-09-09 14:26:26 +00:00
Owner

Closes #32.

  • Search spinner already existed on main (CSS + show/hide in table.js); verified it covers the whole load() cycle including the finally-hidden path.
  • Fixed the filter fetch to request limit=100 (the API's hard maximum) instead of limit=500, which the API silently clamped to 100 anyway - the mismatch meant the fetch window was mislabeled.
  • Added a Performance Notes section to README documenting client-side filtering over the newest 100 pastes, the 100-paste search horizon, and the future server-side search path.

Build + all tests pass (go build ./..., go test ./internal/...).

Closes #32. - Search spinner already existed on main (CSS + show/hide in table.js); verified it covers the whole load() cycle including the finally-hidden path. - Fixed the filter fetch to request limit=100 (the API's hard maximum) instead of limit=500, which the API silently clamped to 100 anyway - the mismatch meant the fetch window was mislabeled. - Added a Performance Notes section to README documenting client-side filtering over the newest 100 pastes, the 100-paste search horizon, and the future server-side search path. Build + all tests pass (go build ./..., go test ./internal/...).
poslop added 1 commit 2026-09-09 14:13:49 +00:00
Search: spinner indicator + performance note on client-side filtering (#32)
CI / docker (pull_request) Skipped
CI / test (pull_request) Successful in 19s
c288fc73a7
- Fix filter fetch to request limit=100 (API max) instead of 500, which
  the API silently clamped, so filtered results actually cover the fetch window.
- Document client-side filtering behavior and limits in README Performance Notes.
poslop merged commit 4cb8a3e127 into main 2026-09-09 14:26:26 +00:00
Sign in to join this conversation.