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

Fixes all contracts to be compatible with the OpenZeppelin V5 update #138

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wyattwestdev
Copy link

This is a fix that updates the LayerZero/solidity-examples code to be compatible to the OpenZeppelin V5 update regarding the Ownable class inheritance. I also recommend the creation of a new tag/release for the repository to help Brownie/Python frontend developers to import LayerZero's library in a more quicker and efficient way (brownie doesn't accept importing libraries without versioning/tag).

… compatible to the OpenZeppelin V5 update regarding the Ownable class inheritance.
@wyattwestdev wyattwestdev reopened this Jan 7, 2024
@wyattwestdev wyattwestdev changed the title Ownable constructor fix regarding the OpenZeppelin V5 update Fixes all contracts to be compatible with the OpenZeppelin V5 update Jan 7, 2024
@wyattwestdev
Copy link
Author

I also added a fix to make all other contracts compatible with the OpenZeppelin V5 update.

@JOY
Copy link

JOY commented Jan 23, 2024

Waiting for the merge. Any update please?

@wyattwestdev
Copy link
Author

Waiting for the merge. Any update please?

Hey, JOY. The team is focused on deploying the LayerZero V2, but as soon as it goes live, they will review this PR.

@JOY
Copy link

JOY commented Jan 29, 2024

Waiting for the merge. Any update please?

Hey, JOY. The team is focused on deploying the LayerZero V2, but as soon as it goes live, they will review this PR.

Yeah It's live :D

@kevupton
Copy link

kevupton commented Feb 1, 2024

looks good, what is eta on this one anyone? 😄

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.

3 participants