connext / connext/chaindata

duplicate functionality?

Open
#7 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Solidity
Stars
166
Forks
175
PR merge metrics
No merged PRs in 30d

Description

Hi - is there a reason theres duplicate functionality? setVault/changeVault seem to do the same thing. ChangeMPCOwner is also really similar, the only difference is that it's a public function instead of external
Link to code:
https://github.com/connext/chaindata/blob/main/AnyswapV4ERC20.sol#L203
https://github.com/connext/chaindata/blob/main/AnyswapV4ERC20.sol#L224
https://github.com/connext/chaindata/blob/main/AnyswapV4ERC20.sol#L232

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.