-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fix VotingPowerForProposalExecution #351
Fix VotingPowerForProposalExecution #351
Conversation
✔️ Preview deployment is ready! 🔨 Explore the source changes: d79ca84 😎 Browse the preview: https://bafybeibahhaoz5iv3wqbkeitwvxpxgtvphytlaoz3x63h3wfqmqnxuftmq.ipfs.cf-ipfs.com |
✔️ Storybook deployment is ready! 😎 Browse Storybook: https://bafybeibaswfrlqtuhfx4ewrn2m7bmdh5u6o3a3cdicryemcimenqex7h4y.ipfs.cf-ipfs.com |
✔️ Preview deployment is ready! 🔨 Explore the source changes: 2903ad6 😎 Browse the preview: https://bafybeicxj7yppmafzj2denxeqxfyde7d4ffznywgbckaaejhlwxn3u7dby.ipfs.cf-ipfs.com |
✔️ Storybook deployment is ready! 😎 Browse Storybook: https://bafybeie7zn5brlv43yw3r62zsgfmf67h4iw7wjejrjnmx6oua5qh2wmzdi.ipfs.cf-ipfs.com |
Looks good 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job on this implementation! 🌟 Just a few comments. Also wondering a bit if you think your validation would help us with null
values in the other hooks?
args: [_proposalId], | ||
}; | ||
// Validate args to avoid null values | ||
if (isRepGuild && isSnapshotGuild && !!_proposalId) return snapshotRepConf; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 I wonder if we could add this to the other hooks as well to avoid having null
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not following. What are "this" and "other hooks"? Not sure what hooks are you referring to. Particularly this hook can have a fallback, but not sure about all of them.
Playing around with this, all quorum percentages and values look great, amazing work! |
4724b58
to
e760993
Compare
✔️ Storybook deployment is ready! 😎 Browse Storybook: https://bafybeihhl4xx2pyjfgx2fbhg4ttmbawrevlbiw5ja7rqek44dtm2cl7s2i.ipfs.cf-ipfs.com |
✔️ Preview deployment is ready! 🔨 Explore the source changes: 33dded6 😎 Browse the preview: https://bafybeied36qlwxuhoemilhgiohjmgo67q6bgtxdh6uool2tonpkxzgtczq.ipfs.cf-ipfs.com |
✔️ Preview deployment is ready! 🔨 Explore the source changes: 147cb72 😎 Browse the preview: https://bafybeig4jt3cxuwdrzwrehbj4d3yggd7jp6buztlwoyirrikkk7keykpsm.ipfs.cf-ipfs.com |
✔️ Storybook deployment is ready! 😎 Browse Storybook: https://bafybeiez4sq6nupzlehlckbnx34gz3o7ftv63k5alvml2ojxwgihlknnf4.ipfs.cf-ipfs.com |
✔️ Preview deployment is ready! 🔨 Explore the source changes: 2f29a01 😎 Browse the preview: https://bafybeigrnnui4tm57odzjhqqrm5d4yzswanseivtjdlqipxuqow4qvsnzi.ipfs.cf-ipfs.com |
✔️ Storybook deployment is ready! 😎 Browse Storybook: https://bafybeicq3srdstir2xofchuxjdru2h7zr3uldg2f3jby5jsnawhumgdkla.ipfs.cf-ipfs.com |
Description
useVotingPowerForProposalExecution
hook to return data according to guild type.useGuildConfig
hook and include data from newuseVotingPowerForProposalExecution
hook.SnapshotERC20Guild
Closes #330
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Checklist: