What happened?
We currently return ERC20InvalidUnderlying { token: contract_address } on error, which is problematic for 2 reasons:
- This doesn't align with Solidity behavior.
- The "invalid underlying token" should be
token: underlying_token, not token: contract_address (i.e. wrapper address).
platform
Expected behavior
Update the Wrapper to return Vec<u8> on all functions, enabling errors bubbling up.
Contribution Guidelines
What happened?
We currently return
ERC20InvalidUnderlying { token: contract_address }on error, which is problematic for 2 reasons:token: underlying_token, nottoken: contract_address(i.e. wrapper address).platform
Expected behavior
Update the Wrapper to return
Vec<u8>on all functions, enabling errors bubbling up.Contribution Guidelines