daostack / arc

Arc is an operating system for DAOs.
GNU General Public License v3.0
47 stars 25 forks source link

Let proposeContributionReward take an array of beneficiaries #440

Open dkent600 opened 6 years ago

dkent600 commented 6 years ago

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 commented 6 years ago

Let proposeContributionReward take an array of beneficiaries

orenyodfat commented 6 years ago

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.

leviadam commented 6 years ago

I don't think this should be under this contract

dkent600 commented 6 years ago

@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)?

leviadam commented 6 years ago

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