Skip to content

Rpc get ledgers - #1231

Merged
Ryang-21 merged 5 commits into
masterfrom
rpc-get-ledgers
Oct 8, 2025
Merged

Rpc get ledgers#1231
Ryang-21 merged 5 commits into
masterfrom
rpc-get-ledgers

Conversation

@Ryang-21

@Ryang-21 Ryang-21 commented Oct 2, 2025

Copy link
Copy Markdown
Contributor
  • Add getLedgers function to the RPC server
  • Move GetEventsRequest to rpc.api file and make start/end ledger and cursor mutually exclusive
  • Add documentation for GetEventsRequest
    Addresses Support the new getLedgers endpoint #944

@github-project-automation github-project-automation Bot moved this to Backlog (Not Ready) in DevX Oct 2, 2025
@Ryang-21
Ryang-21 requested a review from Shaptic October 2, 2025 22:51
@Ryang-21 Ryang-21 moved this from Backlog (Not Ready) to In Progress in DevX Oct 2, 2025
@github-actions

github-actions Bot commented Oct 2, 2025

Copy link
Copy Markdown

Size Change: +176 kB (+0.42%)

Total Size: 42.3 MB

Filename Size Change
dist/stellar-sdk-minimal.js 5.52 MB +28.3 kB (+0.52%)
dist/stellar-sdk-minimal.min.js 4.82 MB +15.8 kB (+0.33%)
dist/stellar-sdk-no-axios.js 5.52 MB +28.3 kB (+0.52%)
dist/stellar-sdk-no-axios.min.js 4.82 MB +15.8 kB (+0.33%)
dist/stellar-sdk-no-eventsource.js 5.77 MB +28.3 kB (+0.49%)
dist/stellar-sdk-no-eventsource.min.js 5.02 MB +15.8 kB (+0.31%)
dist/stellar-sdk.js 5.77 MB +28.3 kB (+0.49%)
dist/stellar-sdk.min.js 5.02 MB +15.8 kB (+0.31%)

compressed-size-action

@Ryang-21 Ryang-21 moved this from In Progress to Needs Review in DevX Oct 3, 2025

@Shaptic Shaptic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work and a clean impl 👍 be sure to add a changelog entry!

Comment thread CHANGELOG.md
@Ryang-21
Ryang-21 merged commit 537e818 into master Oct 8, 2025
10 checks passed
@Ryang-21
Ryang-21 deleted the rpc-get-ledgers branch October 8, 2025 23:05
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in DevX Oct 8, 2025
@Shaptic Shaptic mentioned this pull request Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants