fix: prevent unnecessary fetch when reaching the maximum page #354
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
π Linked issue
resolves #353
β Type of change
π Description
The page change handler prefetches
page.value + 1on each navigation, but itdoes not check whether this next page exceeds
maxPage. This causes the app tofetch non-existent pages (e.g. page 11 when only 10 pages exist), resulting in
unnecessary network requests without any benefit.
This PR updates the logic to:
to + 1) as the prefetch target instead ofpage.value + 1.to + 1 <= maxPage.displayedPageis always updated, even on the last page.This avoids invalid requests, improves runtime efficiency, and preserves the
intended UX behavior.