Skip to content

Add ledger close times and oldest ledger in DB to getEvents response#409

Merged
Shaptic merged 2 commits intoprotocol-23from
moreEventsDetails
Apr 17, 2025
Merged

Add ledger close times and oldest ledger in DB to getEvents response#409
Shaptic merged 2 commits intoprotocol-23from
moreEventsDetails

Conversation

@Shaptic
Copy link
Contributor

@Shaptic Shaptic commented Apr 17, 2025

What

Adds the following to the base-level getEvents response schema to match other endpoints:

	OldestLedger          uint32 `json:"oldestLedger"`
	LatestLedgerCloseTime int64  `json:"latestLedgerCloseTime,string"`
	OldestLedgerCloseTime int64  `json:"oldestLedgerCloseTime,string"`

Why

Closes #231.

Known limitations

n/a

@Shaptic Shaptic requested a review from a team April 17, 2025 04:13
@Shaptic Shaptic merged commit fb33a51 into protocol-23 Apr 17, 2025
10 of 13 checks passed
@Shaptic Shaptic deleted the moreEventsDetails branch April 17, 2025 17:37
@Shaptic Shaptic mentioned this pull request Apr 17, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tamirms tamirms tamirms approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Shaptic @tamirms