Skip to content

ExecuteMsgs Gov Proposal Type - #66

Draft
sunnya97 wants to merge 1 commit into
modular-gov-tallyfrom
ExecuteMsg-gov-content
Draft

ExecuteMsgs Gov Proposal Type#66
sunnya97 wants to merge 1 commit into
modular-gov-tallyfrom
ExecuteMsg-gov-content

Conversation

@sunnya97

@sunnya97 sunnya97 commented Feb 4, 2021

Copy link
Copy Markdown

Description

Adds a new governance proposal type that executes Msgs from a specific address. Tries to use the TallyStrategy of the address if registered. Otherwise falls back to the "root" strategy.

closes: #XXXX


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer
  • Review Codecov Report in the comment section below once CI passes

@sunnya97
sunnya97 marked this pull request as draft February 4, 2021 10:35
@sunnya97
sunnya97 changed the base branch from master to modular-gov-tally February 4, 2021 10:35
Comment thread x/gov/types/proposal.go
// TallyRoute is the required msgs Signer
func (emp *ExecuteMsgsProposal) TallyRoute() string {
msgs := sdk.DecodeMsgs(emp.Messages)
return sdk.GetSignersMsgs(msgs)[0].String()

@antstalepresh antstalepresh Feb 5, 2021

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

message signer itself is a tally router?

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