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.

Sign up for GitHub

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

Jump to bottom

Fixed issues with the interface contracts #389

Open

Amtaf wants to merge 1 commit into Uniswap:main from Amtaf:my-fix

Amtaf commented Jun 10, 2024

The Solidity files in your project are using outdated import paths for OpenZeppelin libraries,Specifically in the "https://github.com/Uniswap/v3-periphery/blob/main/contracts/interfaces/INonfungiblePositionManager.sol". Specifically, the import statements reference paths that are not compatible with the latest version of OpenZeppelin. For instance, the imports:

import '@openzeppelin/contracts/token/ERC721/IERC721Metadata.sol';
import '@openzeppelin/contracts/token/ERC721/IERC721Enumerable.sol';

The incorrect import paths caused the following issues:

Compilation Errors: The contracts fail to compile, halting the development and deployment process.
Development Delay: Additional time is required to identify and correct the import paths, leading to potential delays in project timelines.
Integration Problems: Projects integrating with libraries or contracts expecting the correct paths will face compatibility issues.


          Fixed issues with the interface contracts

d10d7b9

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