-
Notifications
You must be signed in to change notification settings - Fork 7
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
Remove linear erc721 voting #1669
Conversation
✅ Deploy Preview for fractal-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
👍
tokenId, | ||
); | ||
const tokenVoted = await votingContract.read.hasVoted([ | ||
Number(_proposalId), |
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.
Can proposal Id ever be anything but a number? possibly so? just wanted to ask anyway
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.
Good call out. Found some bugs here. Fixing now.
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.
Well, fixed the bugs (5fade89).
I also moved where this cast was happening, but TL;DR is that by the time this code here is executing, _proposalId
, although a string, is a number. I certainly don't love this. The deeper issue is that we're sharing so many types/interfaces between erc20/erc721/multisig/snapshot proposals, when we really shouldn't be.
5ce6725
to
97c6677
Compare
4e763e6
to
0592a19
Compare
e9b9a7f
to
e8a48a7
Compare
0592a19
to
128214a
Compare
e8a48a7
to
5dafa68
Compare
128214a
to
afb326c
Compare
5dafa68
to
7757f01
Compare
afb326c
to
143efd2
Compare
7757f01
to
6f80110
Compare
143efd2
to
060a635
Compare
e13ad9f
to
ae94995
Compare
060a635
to
bef29c0
Compare
…UserSubmitProposal
…721LinearStrategy
…oriusContracts, useSafeContracts
bef29c0
to
bfcaeac
Compare
…the function is being called, proposalId must be a numberish
Please review and merge #1668 first.