Skip to content
This repository was archived by the owner on Sep 9, 2025. It is now read-only.

Conversation

@fragchain
Copy link

I noticed that the function _getSalt was being called with named arguments ({config: config, salt: salt}), which is not supported in Solidity. This would cause a compilation error. I've updated the code to pass the arguments in the correct order, as required by Solidity. The corrected line now looks like this:

(bool alreadyDeployed, address accountAddress) =
    LibClone.createDeterministicERC1967(msg.value, implementation, _getSalt(config, salt));

This change ensures the code compiles without errors.

@fragchain
Copy link
Author

RainbowScientist5 now you can merge it :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants