terra-money / alliance

🤝 Alliance chain with x/alliance module
Other
49 stars 31 forks source link

Allow asset participation in governance to be configurable #50

Open javiersuweijie opened 1 year ago

javiersuweijie commented 1 year ago

Not all assets should be allowed to participate in governance as there might be conflict of interests in some cases.

javiersuweijie commented 1 year ago

An alternative solution is to make sure that the alliance module votes abstain on all proposals that are IN_VOTING this way the alliance will not be able to affect the results of a vote.

This solution loses the ability to allow individual alliances to be configured with governance and also might potentially allow all proposal to pass quorum even when there are low voter participation.

phamminh0811 commented 1 year ago

Should we configure this module in a fork version or directly add a gov module into x folder?

emidev98 commented 1 year ago

@phamminh0811 the module can be created inside the /custom folder from alliance module, similar to the bank module implementation and it can be merged into the main branch

phamminh0811 commented 1 year ago

great, I'm working on that