-
Notifications
You must be signed in to change notification settings - Fork 118
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
don't play well with openzepplin #68
Comments
@cedricwalter thanks for the report -- I've noticed you closed this ticket -- did you find a solution? any updates on your experience? thanks |
Yes and no, I switched to a pure Open Zeppelin ERC1155 implementation. I closed this report since it is not really an error, mixing 2 library (yours and open zeppelin), and adding truffle is something you may not want to urgently fix. |
Cool thanks. I'll reopen for us to look into some time in the future |
I actually prefer this implementation but want to use some other contracts from Open Zeppelin. I run into this same issue. Are you planning to upgrade to 0.8.*? |
@aalmada yes we will – it will be an easy upgrade too. curious which parts you prefer in this implementation? |
Thanks for this great library! but updating openzeppelin-solidity to latest 4.0.0 and I can not compile my project
running but outdated combination taken from https://github.com/ProjectOpenSea/opensea-erc1155
How to reproduce, upgrading 3rd parties
Tried both
or
what do you recommend? rollbacking openzeppelin-solidity to a previous version? hoping that I find one which stilluse Solc 0.7 branch?
The text was updated successfully, but these errors were encountered: