This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

Remove superfluous ERC20 address argument from contract deposit function #971

Open

sveitser opened this issue Apr 19, 2022 · 1 comment

Labels

contract priority: low

Contributor

sveitser commented Apr 19, 2022

Follow up on

Deposit function does not check if ERC20 address matches that of registered asset definition #954

It's not necessary to pass the token address, we can get it from the registry.

This will change the contract ABI so will require quite a bit of changes in test code and frontend.

sveitser added the contract label

sveitser mentioned this issue

Check ERC20 address matches registered address #968

Merged

Contributor Author

sveitser commented Apr 20, 2022

This is a low priority task at the moment.

sveitser added the priority: low label

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.