Skip to content

Add pagination to invoice list queries.#122

Open
ndyugwu wants to merge 6 commits into
TrusTrove:mainfrom
ndyugwu:issue-71-add-pagination
Open

Add pagination to invoice list queries.#122
ndyugwu wants to merge 6 commits into
TrusTrove:mainfrom
ndyugwu:issue-71-add-pagination

Conversation

@ndyugwu

@ndyugwu ndyugwu commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Bounds gas per call by accepting page and page_size on get_by_status, get_by_issuer, and get_by_buyer.
close #71

Bounds gas per call by accepting page and page_size on get_by_status, get_by_issuer, and get_by_buyer.

Co-authored-by: Cursor <cursoragent@cursor.com>
@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

@ndyugwu Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

ndyugwu and others added 5 commits June 28, 2026 14:26
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Align test snapshots with paginated get_by_* signatures and remove dedicated pagination tests that lacked snapshot files.

Co-authored-by: Cursor <cursoragent@cursor.com>
@K1NGD4VID

Copy link
Copy Markdown
Contributor

Please Resolve conflict @ndyugwu

@K1NGD4VID

Copy link
Copy Markdown
Contributor

resolve merge conflict @Cjay-Cyber-2

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.

get_by_status/get_by_issuer/get_by_buyer lack pagination, risk hitting Soroban budget limits

3 participants