Skip to content

Conversation

@sullof
Copy link
Contributor

@sullof sullof commented Sep 29, 2023

The examples in the repo, despite being basic, are correct and safe and can be used as a solid base to build more advanced contracts. In order to do so, the functions must be virtual, so that they can be overridden.

@jaydenwindle This PR adds virtual to the functions' signatures, making them also public so they can be called as super.

@jaydenwindle
Copy link
Contributor

@sullof thanks for the PR!

@jaydenwindle jaydenwindle merged commit 1309fb5 into erc6551:main Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants