Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

update getLedgerEntries documentation #614

Conversation

tsachiherman
Copy link
Contributor

What ?

Following up on stellar/stellar-cli#916, this PR updates the corresponding documentation for the getLedgerEntries method.

@tsachiherman tsachiherman self-assigned this Oct 16, 2023
@stellar-jenkins
Copy link

- `key`: `<xdr.LedgerKey>` - The key of the ledger entry (serialized in a base64 string).
- `xdr`: `<xdr.LedgerEntryData>` - The current value of the given ledger entry (serialized in a base64 string).
- `lastModifiedLedgerSeq`: `<string>` - The ledger number of the last time this entry was updated.
- `expirationLedgerSeq`: `<number>` - The ledger sequence number after which the ledger entry would expire. This field exists only for ContractCodeEntry and ContractDataEntry ledger entries (optional).
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be a <string>, it has same go json string qualifier on the struct as lastModifiedLedgerSeq which shows <string> type here:

https://github.com/stellar/soroban-tools/pull/916/files#diff-50af22ea9690b67db00854506e8b1b086648e3d5bee1d8cdd2a1bb6f409c741cR31

@stellar-jenkins
Copy link

@tsachiherman tsachiherman merged commit 7506f2b into stellar-deprecated:main Oct 16, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants