Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add initialOwner to TeleporterOwnerUpgradeable #262

Closed
minghinmatthewlam opened this issue Jan 30, 2024 · 0 comments · Fixed by #265
Closed

Add initialOwner to TeleporterOwnerUpgradeable #262

minghinmatthewlam opened this issue Jan 30, 2024 · 0 comments · Fixed by #265
Assignees
Labels
enhancement New feature or request

Comments

@minghinmatthewlam
Copy link

Context and scope
We currently use openzeppelin contracts v4.8.1 because the newer releases require solc version ^0.8.20, but Teleporter had its audits on version 0.8.18 and don't think it's worth the changes to update audited code right before launch.

Discussion and alternatives
In comparing the differences between OZ's latest release compared to our set release version, there weren't too many notable changes for code we import from OZ. This issue does address a change to add initialOwner as an argument to Ownable so that at deployment we can set an owner different from deployer without the additional need to call transferOwnership.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant