Skip to content

Expose blockchain heights#7

Open
elusiveshiba wants to merge 9 commits into
mainfrom
feature/blockchain-heights
Open

Expose blockchain heights#7
elusiveshiba wants to merge 9 commits into
mainfrom
feature/blockchain-heights

Conversation

@elusiveshiba

@elusiveshiba elusiveshiba commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

This exposes the underlying Core sync heights by updating the /height endpoint to include coreBlocksHeight , coreHeadersHeight and coreSyncUpdatedAt

@elusiveshiba elusiveshiba changed the title Blockchain heights Expose blockchain heights Jun 2, 2026
@elusiveshiba elusiveshiba requested a review from raffecat June 2, 2026 00:15
@elusiveshiba elusiveshiba marked this pull request as ready for review June 2, 2026 00:15

@raffecat raffecat left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks OK – not sure if suppressing the sync data when stale was intentional.

Comment thread web/sync_heights.go Outdated
Comment thread main.go Outdated
Comment thread main.go Outdated

@raffecat raffecat left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Comment

Replace spec.CoreRequestClient with dogewalkerspec.Blockchain and GetBlockchainInfo
Remove now redundant spec/core.go interface.
…blockchain-heights. Also rename dogewalkerspec to walkerspec for alignment with existing vars

# Conflicts:
#	main.go
@elusiveshiba elusiveshiba requested a review from raffecat June 11, 2026 04:28
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.

2 participants