BreadchainCoop / breadchain

2 stars 1 forks source link

Butter that Bread #78

Closed daopunk closed 1 month ago

daopunk commented 1 month ago

closes #73

RonTuretzky commented 1 month ago

Due to the way the voting delay works, syncing this at the time or voting will not configure the voting power for that cycle, only the next one @daopunk ... Not sure if this is a good solution or not .

I feel maybe when we change the scaling factor, we should submit a list of holders alongside the change and sync them as you've implemented

daopunk commented 1 month ago

I feel maybe when we change the scaling factor, we should submit a list of holders alongside the change and sync them as you've implemented

Yes, makes sense. Well I think it's still good to update active voters in the next cycle. Plus, there is a batch-sync function that can be called with a list of users. Likewise, they will automatically update during withdrawals of any amount. So it's fairly flexible.