-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update getEvents docs #275
Labels
Milestone
Comments
Maybe we defer this change until p22, since @aditya1702 is planning on testing up to 7 days at a later time |
I think this will need to wait since we are not releasing all the breaking changes in P22 now (due to the SDK backward compatibility issues). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Update getEvents doc based on #230
EndLedger
inGetEventsRequest
. This provides finer control and clarity on the range of ledgers being queried.10,000
ledgers per request. This limits the resource usage and ensures more predictable performance, especially for queries spanning large ledger ranges.The text was updated successfully, but these errors were encountered: