Skip to content

consensus/parlia: add GetJustifiedNumber and GetFinalizedNumber#2591

Merged
zzzckck merged 1 commit intobnb-chain:developfrom
buddh0:parlia_api_get
Jul 19, 2024
Merged

consensus/parlia: add GetJustifiedNumber and GetFinalizedNumber#2591
zzzckck merged 1 commit intobnb-chain:developfrom
buddh0:parlia_api_get

Conversation

@buddh0
Copy link
Copy Markdown
Contributor

@buddh0 buddh0 commented Jul 17, 2024 •
edited
Loading

Description

consensus/parlia: add GetJustifiedNumber and GetMinerHistory

Rationale

parlia_getJustifiedNumber

curl http://127.0.0.1:8545/ \
	-X POST \
	-H "Content-Type: application/json" \
	--data '{"method":"parlia_getJustifiedNumber","params":["0x116"],"id":1,"jsonrpc":"2.0"}'

{"jsonrpc":"2.0","id":1,"result":277}

parlia_getFinalizedNumber

curl http://127.0.0.1:8545/ \
	-X POST \
	-H "Content-Type: application/json" \
	--data '{"method":"parlia_getFinalizedNumber","params":["0x116"],"id":1,"jsonrpc":"2.0"}'

{"jsonrpc":"2.0","id":1,"result":276}

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

galaio
galaio reviewed Jul 18, 2024
View reviewed changes
@buddh0 buddh0 changed the title consensus/parlia: add GetJustifiedNumber and GetMinerHistory consensus/parlia: add GetJustifiedNumber and GetFinalizedNumber Jul 18, 2024
@zzzckck
Copy link
Copy Markdown
Collaborator

zzzckck commented Jul 19, 2024

add some background:
this is for some test case to make it easier to check the network stabillity around some block height.

@zzzckck zzzckck merged commit 74078e1 into bnb-chain:develop Jul 19, 2024
This was referenced Aug 7, 2024
Merged
Merged
@tak1827 tak1827 mentioned this pull request Nov 6, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@galaio galaio galaio approved these changes

@zlacfzy zlacfzy zlacfzy 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.

4 participants

@buddh0 @zzzckck @galaio @zlacfzy