Skip to content
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.

Sign up for GitHub

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

Update VotesERC20Wrapper #1638

Merged
merged 19 commits into from
May 5, 2024
Merged

Update VotesERC20Wrapper #1638

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
7e05b91
DRY chain references in network configs
adamgall May 4, 2024
dc4c650
Change type of VotesERC20Wrapper to Address, use the address from new…
adamgall May 4, 2024
5ea9fc1
Install new version of contracts repo alongside existing version
adamgall May 4, 2024
6b6f46d
Add "as const" to GnosisSafeL2 abi, and therefore fix some types
adamgall May 5, 2024
7315571
Add an ABI for VotesERC20
adamgall May 5, 2024
d942e11
Create a second useTransaction hook for these viem-style contract calls
adamgall May 5, 2024
c4518f4
Add type to declaration, not "as type" when instantiating
adamgall May 5, 2024
78379cb
Don't need walletClient, only reading
adamgall May 5, 2024
df5fff9
I think the "wrong" type was being used on this address, but it was f…
adamgall May 5, 2024
ce17762
Add an ABI for VotesERC20Wrapper
adamgall May 5, 2024
f81960d
Remove new fractal-contracts package, that didn't work as I hoped it …
adamgall May 5, 2024
6fdb75a
Remove votesERC20WrapperMasterCopyContract from AzoriusContracts type…
adamgall May 5, 2024
2a4304a
Remove VotesERC20WrapperMasterCopyContract "asSigner / asProvider" ob…
adamgall May 5, 2024
3984d1c
Fix all of the typescript errors from previous two commits
adamgall May 5, 2024
8531ef7
Remove unnecessary reference to VotesERC20Wrapper export from 'fracta…
adamgall May 5, 2024
8d62449
Remove VotesERC20 type, save this for another PR
adamgall May 5, 2024
1fc3d78
Standardize local Abi import naming
adamgall May 5, 2024
195070f
Don't need to check type any longer
adamgall May 5, 2024
273235d
Fix Hash => Hex types
adamgall May 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions src/assets/abi/GnosisSafeL2.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
export default [
const GnosisSafeL2Abi = [
{
anonymous: false,
inputs: [
Expand Down Expand Up @@ -1135,4 +1135,6 @@ export default [
stateMutability: 'payable',
type: 'receive',
},
];
] as const;
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding as const here is necessary for viem's typescript support.

https://viem.sh/docs/typescript#type-inference

This small change resulted in some type cleanup in the one place where this ABI was being used!


export default GnosisSafeL2Abi;
Loading