Skip to content

fix: Iter.Reload returns an already-consumed iterator and drops list params - #9

Open
childish-sambino wants to merge 1 commit into
Files-com:masterfrom
childish-sambino:fix/iter-reload
Open

fix: Iter.Reload returns an already-consumed iterator and drops list params#9
childish-sambino wants to merge 1 commit into
Files-com:masterfrom
childish-sambino:fix/iter-reload

Conversation

@childish-sambino

Copy link
Copy Markdown

Reload copied the consumed iterator (Values, CurrentIndex, Page, Cursor) and replaced ListParams with a bare &ListParams{}. A fully-iterated list reloads to zero items, since GetPage bails when Values is set and Cursor is empty, and PerPage plus every typed filter param was dropped. Every generated resource client wraps this method verbatim.

Reload now resets the iteration state and keeps the original params, so it re-lists from page one with the same filters.

🤖 Generated by Quad tha God

@childish-sambino
childish-sambino marked this pull request as ready for review July 7, 2026 22:26
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