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

VoteTx should fetch all the voter delegations on wasm #2523

Closed
Tracked by #2529
Fraccaman opened this issue Feb 5, 2024 · 2 comments · Fixed by #3120
Closed
Tracked by #2529

VoteTx should fetch all the voter delegations on wasm #2523

Fraccaman opened this issue Feb 5, 2024 · 2 comments · Fixed by #3120

Comments

@Fraccaman
Copy link
Member

Fraccaman commented Feb 5, 2024

Right now, the vote_proposal tx is expecting from the client a list of validators for which the voter has a delegation to. The Governance VP is then validating the list, but the validation logic is redundant and not optimal. We should instead fetch the voter delegation list directly from the wasm transactions.

@Fraccaman
Copy link
Member Author

@brentstone it would come real handy if we could easily fetch the set of validator addresses for a specific delegator

@brentstone
Copy link
Collaborator

If I recall correctly, there is a dependency issue between proof-of-stake and governance here that would possibly require a refactor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants