Logflare / logflare / 35267251088-1

83%

main: 83%

Build Type

Pull #3984

github

Committed by chasers

Commit Message

fix: address the review of the load more stack

- Store one page window per search in `EventPagination`. The label, the
  query, the cursor shift and the range growth all read it. The window no
  longer doubles on each click.
- Set the page window from a tail result too. `resume_tailing/1` resets the
  pagination, and the result that follows takes the tail path, so a soft
  pause, play and pause left the window nil and dropped the next page
  request. The window belongs to the search in view, not to the first page.
- Build the chart range in the search timezone. An implied range is no
  longer UTC written as local time.
- Keep a one-sided `t:` bound when a page request makes the range explicit.
- Cap the cursor of an empty "next" page at the request time.
- Clamp a "next" range extension at now. Each empty click used to push the
  range max another window into the future and re-run the aggregate.
- Drop a page result or page error that no request waits for.
- Clear the page spinner when a search fails to parse.
- Disable the other button while a page request runs.
- Remove the unused sentinel row: the extra fetched row, `fetch_limit/0`
  and the `has_more?` field that nothing reads.
- Remove dead clauses, a duplicate alias and inline comments.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YBPF1knvgfghTSdxuYryGL

Pull Request Pull Request #3984: fix: address the review of the load more stack

Coverage Stats

78 of 85 new or added lines in 7 files covered. (91.76%)

3 existing lines in 2 files now uncovered.

15561 of 18680 relevant lines covered (83.3%)

1973.01 hits per line