arc icon indicating copy to clipboard operation
arc copied to clipboard

Let proposeContributionReward take an array of beneficiaries

Open dkent600 opened this issue 6 years ago • 5 comments

Let ContributionReward.proposeContributionReward take an array of beneficiaries. This would be useful for voting, in a single proposal, to bring in large groups of DAO participants all at once who aren't founders.

dkent600 avatar Apr 18 '18 18:04 dkent600

If you are looking for a way to add participants to a dao you can register a scheme to the controller and mint reputation for each of the participants..no need the contributionReward scheme for that..

If you suggest to conduct a voting to mint reputation/tokens for a set of beneficiaries ...it is another story. ContributionReward scheme is currently a binary voting (yes/no) ... Modify it to take an array of beneficiaries (actually it will be an array of proposals), and conduct a binary voting on that ...might be a bit difficult and complex for the voters to vote and give the attention for each beneficiary though it might be possible. I am not sure about modifying ContributionReward . How about writing additional scheme ,if there is a real need for that.

orenyodfat avatar Apr 21 '18 15:04 orenyodfat

I don't think this should be under this contract

leviadam avatar Apr 22 '18 11:04 leviadam

@leviadam

I don't think this should be under this contract

I'm interested in your reasoning on this. Further, are you suggesting that, even with the gas limitations, this contract is worth having as a separate contract (don't worry, it won't hurt my feelings if you say no)?

dkent600 avatar Apr 22 '18 18:04 dkent600

It is easy to have a beneficiary contract that splits the reward. And we should modularize and make logic simple when possible.

leviadam avatar Jul 22 '18 13:07 leviadam