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

Submission Week 4 Assignment #10

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

Conversation

VarshiniAdurti28
Copy link

My attempt was to solve the errors in the present code and try to make it work:
So, I tried to solve all the errors that the files initially had: (put the errors {the way I found them and then the solution} in comments for my future reference)

  • created account on Alchemy API and got my API key for Sepolia testnet.
  • added hardhat.config.ts (adding the chainId of sepolia helped solve some transaction issues)
  • made some changes in compilerOptions in tsconfig.json
  • changed some variable names and made them uniform across all files.
  • deployed the contract on hardhat and got the contract address added it onto the .env file
  • then I minted the NFT using the command given (MyNFT.sol used Counters.sol which was deprecated in the newer versions according to stack overflow (but I am not sure) so I wrote code for a simple counter.
  • coming to the test files, this is where I encountered too many errors: mostly dependency issues
  • I then degraded all ethers versions to 5.7.2 because that was uniform across more than 2 files.
  • I solved a few more errors in tester files.
  • All errors resolved (as in, on vs code) but npx hardhat test gives the following error as a result of conflict of type: commonJS and ESM module under compiler options of tsconfig.json which I was not able to resolve. (Image attached)
  • Any feedback or suggestions are welcome.
    Screenshot 2024-10-24 143553

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