8000 Heimdallv2 final by avalkov · Pull Request #1574 · maticnetwork/bor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Heimdallv2 final #1574

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 192 commits into from
Jun 18, 2025
Merged

Heimdallv2 final #1574

merged 192 commits into from
Jun 18, 2025

Conversation

avalkov
Copy link
Contributor
@avalkov avalkov commented Jun 12, 2025

Description

All the required changes for the heimdall-v2 release.

Raneet10 and others added 30 commits December 24, 2024 18:32
@Raneet10
cmd,consensus/bor,eth,internal,tests: init heimdall-v2 integration
aab0f8b
@Raneet10
cmd,consensus/bor,eth,internal: refactor and todo comments
01c4315
@Raneet10
go mod tidy
280e0af
@Raneet10
update cosmos-sdk/client/v2
bd5eddf
@Raneet10
consensus/bor: fix setting of proto message value to correct type
d372b13
@avalkov
Fix milestone response
7e465cb
@avalkov
Latest milestone response fixes
eba691c
@avalkov
Milestone custom unmarshal
0ab6c16
@avalkov
Fix milestone parsing
402c030
@avalkov
Fix milestone parsing
14edef7
@avalkov
Fix milestone no-ack parsing
7fbf825
@avalkov
Fix milestone no-ack parsing
63050a0
@avalkov
Remove log
8ceff9a
@avalkov
Test fixes
b0e57c5
@avalkov
Merge pull request #1399 from maticnetwork/raneet10/heimdallv2-change…
46b1b62 
…s-request-responses-fixes

Fix responses and parsing for milestone related requests.
@pratikspatil024
checkpoint: fix response
6a2f587
@avalkov
Custom unmarshaling for checkpoint
9295e5f
@avalkov
Fix fetching milestone by id
0dfd95d
@avalkov
Fix milestone count response
a90857e
@avalkov
Fix checkpoint responses
ca91db6
@avalkov
Remove FetchMilestoneByID
8309852
@pratikspatil024
Merge pull request #1408 from maticnetwork/fix-checkpoint-parsing
0b5bbd0 
HV2 - checkpoint, milestone: fix response
@avalkov
Fix MinimalVal
91383f1
@avalkov
Merge pull request #1414 from maticnetwork/avalkov/Fix-MinimalVal-Ass…
e5bf9cc 
…igment

Fix minimal val assignment
@Raneet10
consensus/bor: add page param to state sync fetch url
2dbee79
@Raneet10
consensus/bor: fix state sync url
6a40234
@Raneet10
consensus/bor: add debug log for state sync
745f852
@Raneet10
consensus/bor: minor change to fetchStateSyncEventsFormat
a2a1e8c
@Raneet10
consensus/bor: fix state sync event record decoding
d3fc673
@Raneet10
update heimdall-v2 commit
c80c9b1
eastorski
eastorski reviewed Jun 13, 2025
View reviewed changes
@avalkov avalkov force-pushed the heimdallv2-final branch 3 times, most recently from c89b413 to ef27db8 Compare June 13, 2025 13:47
@avalkov avalkov force-pushed the heimdallv2-final branch from ef27db8 to 3c77074 Compare June 13, 2025 13:54
manav2401
manav2401 reviewed Jun 16, 2025
View reviewed changes
@avalkov avalkov force-pushed the heimdallv2-final branch from 867b835 to 063681e Compare June 16, 2025 16:40
@avalkov avalkov force-pushed the heimdallv2-final branch from 063681e to 93a0f21 Compare June 16, 2025 17:15
@avalkov avalkov force-pushed the heimdallv2-final branch from 5f687e6 to 14201d8 Compare June 18, 2025 13:44
pratikspatil024
pratikspatil024 reviewed Jun 18, 2025
View reviewed changes
@avalkov avalkov force-pushed the heimdallv2-final branch from 14201d8 to 448d8f9 Compare June 18, 2025 14:11
cffls
cffls approved these changes Jun 18, 2025
View reviewed changes
Copy link
Contributor
@cffls cffls left a comment

Choose a reason for hiding this comment

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

LGTM

@pratikspatil024 pratikspatil024 added the squash and merge This PR will be squashed and merged label Jun 18, 2025
@pratikspatil024 pratikspatil024 merged commit 3dc7d31 into develop Jun 18, 2025
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eastorski eastorski eastorski left review comments

@manav2401 manav2401 manav2401 left review comments

@pratikspatil024 pratikspatil024 pratikspatil024 left review comments

@marcello33 marcello33 marcello33 approved these changes

@cffls cffls cffls approved these changes

Assignees
No one assigned
Labels
squash and merge This PR will be squashed and merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@avalkov @eastorski @marcello33 @manav2401 @pratikspatil024 @cffls @Raneet10 @lucca30 @kamuikatsurgi
0