Skip to content
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.

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

bep173: enable text proposal on BNB Smart Chain #892

Merged
merged 4 commits into from
Nov 21, 2022

Conversation

randyahx
Copy link
Contributor

@randyahx randyahx commented Nov 17, 2022

Motivation

In some scenarios, the community may need a proposal that does not directly cause any changes, like agreeing to a certain strategy, plan, commitment or other statement.

Specification

Right now, there are many system parameters that control the behavior of the BSC, e.g. slash amount, cross-chain transfer fees. All these parameters can be determined by the BSC Validator Set together through a proposal-vote process based on staking. The process executes on the BC and the new parameter values will be picked up by either the management module on the BC or corresponding system contracts via the cross-chain communication protocol. The proposals can be classified into two groups:

  1. Param Change Proposal if the parameter takes effect on the Beacon Chain;

  2. Cross Param Change Proposal if the parameter takes effect on the BNB Smart Chain.

This proposal introduces a new kind of proposal to agree on a certain strategy, plan, commitment, future upgrade, or other statements. Text proposals are exclusively a signaling mechanism and focal point for future coordination - they do not directly cause any changes on-chain.

Commands:

Submit Text Proposal

./tbnbcli gov submit-proposal --type text …[flags]

Query Proposal

./tbnbcli gov query-proposal --proposal-id {id} ...[flags]

Voting

./tbnbcli gov vote --option yes --proposal-id {id} ...[flags]

@randyahx randyahx changed the title bep154: enable text proposal on BNB Smart Chain bep173: enable text proposal on BNB Smart Chain Nov 17, 2022
@@ -60,6 +60,8 @@ BEP128Height = 23551600
BEP151Height = 28250000
#Block height of BEP153 upgrade
BEP153Height = 30516226
#Block height of BEP173 upgrade
BEP173Height = 30516226
Copy link
Collaborator

Choose a reason for hiding this comment

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

need to be decide

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please change to max.

cmd/gen_devnet/main.go Outdated Show resolved Hide resolved
@pythonberg1997
Copy link
Contributor

Please delete the genDevnet script. And the pr's description needs to be changed. What's --home ~/go/src/github.com/bnb-chain/node/build/devnet/node0/testnodecli?

@unclezoro unclezoro added r4r ready for review enhancement New feature or request labels Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request r4r ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants