Skip to content

Commit

Permalink
Merge pull request #30 from uprendis/feature/adjust-network-parameter…
Browse files Browse the repository at this point in the history
…-proposal

Fix scales for NetworkParameterProposal
  • Loading branch information
uprendis authored Nov 12, 2022
2 parents 4b75962 + 755e588 commit 6d2b1fc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion contracts/proposal/NetworkParameterProposal.sol
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ contract NetworkParameterProposal is DelegatecallExecutableProposal, Cancelable
_minEnd = __minEnd;
_maxEnd = __maxEnd;
getOptionVal = __optionsVals;
_opinionScales = [0, 2, 3, 4, 5];
_opinionScales = [0, 1, 2, 3, 4];
consts = ConstsI(__consts);
// verify the proposal right away to avoid deploying a wrong proposal
if (verifier != address(0)) {
Expand Down
2 changes: 1 addition & 1 deletion test/Governance.js
Original file line number Diff line number Diff line change
Expand Up @@ -410,7 +410,7 @@ contract('Governance test', async ([defaultAcc, otherAcc, firstVoterAcc, secondV
return consts;
};

const createNetworkParameterProposalViaFactory = async (factory, _exec, optionsNum, minVotes, minAgreement, startDelay = 0, minEnd = 120, methodID, maxEnd = 1200, _scales = scales) => {
const createNetworkParameterProposalViaFactory = async (factory, _exec, optionsNum, minVotes, minAgreement, startDelay = 0, minEnd = 120, methodID, maxEnd = 1200, _scales = [0, 1, 2, 3, 4]) => {
if (await this.verifier.exists(6003) === false) {
await this.verifier.addTemplate(6003, 'NetworkParameterProposal', emptyAddr, _exec, ratio('0.0'), ratio('0.0'), _scales, 0, 100000000, 0, 100000000);
}
Expand Down

0 comments on commit 6d2b1fc

Please sign in to comment.