Skip to content

Conversation

@Abdkhan14
Copy link
Contributor

@Abdkhan14 Abdkhan14 commented Dec 9, 2025

Behaviour (check out the comment on the endpoint):

  • We fetch 24 rows and accumulate the data across pages
  • Display 12 charts per page from the accumulated data

We maintain both a cursor and a page number state to achieve this

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 9, 2025
@Abdkhan14 Abdkhan14 marked this pull request as ready for review December 10, 2025 20:00
@Abdkhan14 Abdkhan14 requested a review from a team as a code owner December 10, 2025 20:00
Comment on lines +43 to +49
if (
previousSubstringMatch !== substringMatch ||
previousQueryString !== queryString ||
previousPageFilters !== pageFilters
) {
accumulatedDataRef.current = {};
}

This comment was marked as outdated.

@Abdkhan14 Abdkhan14 merged commit e39f402 into master Dec 11, 2025
50 checks passed
@Abdkhan14 Abdkhan14 deleted the abdk/explore-attr-breakdowns-pagination branch December 11, 2025 18:23
@github-actions github-actions bot locked and limited conversation to collaborators Dec 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants