Closed 2xic closed 6 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
evm-codes | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 18, 2024 10:25am |
This stack of pull requests is managed by Graphite. Learn more about stacking.
Join @2xic and the rest of your teammates on Graphite
I just deleted one character from the contract address, leading to this overlap:
Ideally, the MUI inputs can be replaced with already existing Input
too at some point for more consistency.
Ideally, the MUI inputs can be replaced with already existing
Input
too at some point for more consistency.
💯 Yeah, I agree with this. I wanted to get it done with this PR, but it would go past the allowed time for this cleanup :'(
Some polish of the contract viewer
Still some known issue that we will likely tackle later
error handling: if you try to read from a mapping we can't decode it and it will give an error.
you can click on the input boxes of results node, but they aren't editable
the encoded results could have been padded to better align (see photo - 4bytes are unaligned with the rest, we could have just padded the 4 bytes with empty space to make it all aligned)
Some minor styling changes here and there for the various sections inside the function interactions.
Last but not least, there are some improved that could be done to make the code more readable and reusable
Test plan