Lesson 8. Insufficient funds warning triggered

[pacelliv]

I got a massive error when I clicked the fund button. I ran the new Fund Me and Price Converter contracts on Remix to see if there were errors but I called the fund and withdraw functions with no problems.

Now I remember that in Remix the funds were in wei, but now I'm trying to fund in ETH, shouldn't I change something in the fund me contract to match the units?

Lesson 7 repo: https://github.com/PacelliV/hardhat-fund-me
Lesson 8 repo: https://github.com/PacelliV/html-fund-me

Error:

[krakxn]

What are you running? Is it a Unit Test?

[krakxn]

Debugged your L7 repo so far, going to debug L8 now.

[krakxn]

L8 done as well.

The logic should be correct now after merging the PRs.

In your L7 repo, do the following after merging the PRs:

• delete node_modules
• yarn hardhat clean
• yarn install
• yarn hardhat compile

In your L8 repo:

The only possibility is constants.js; copy the one from the course's repo to eliminate such a possibility.

Overall:

The PRs and the above should fix the issue. If not, make sure you have sufficient funds and/or make sure the account is correct.

[pacelliv]

@krakxn I spent last night reading your changes and learning how to use a few new git commands. I made a few mistakes and created new errors trying to fix the mess.

I deleted the folder locally and clone it to start again, I will repeat your steps and this time I'll ask questions.

I'm sorry for dragging this for too long, so far in the course I've been able to debug and fix any error by myself, but this time it's been a nightmare.

[pacelliv]

I'm getting this error again:

I found how to install the plugin here: https://hardhat.org/hardhat-runner/plugins/nomicfoundation-hardhat-toolbox

Last night I ran the yarn method, I copy-pasted the command and then I compiled the contract. But it created a conflict between plugins. The solution recommends to eliminate a list of plugins that we have been working with and I'm confused if I should delete them or not to resolve the conflict.

[krakxn]

hardhat-toolbox is redundant as we already have the packages it includes.

[alymurtazamemon]

@pacelliv Try changing the chain id to 1337 in metamask

[pacelliv]

@alymurtazamemon I changed the chainId to 1337 but the transaction failed, I reset the account and tried again but the result was the same.

I restored the chainId back to 31337, called the fund function, it triggered the insufficient funds warning but the transaction was successful.

The only explanation I can think of is that there is something wrong in my FundMe contract. Maybe after styling and optimizing it in Lesson 7, I made a typo resulting in an error in the abi I'm importing.

I'll try Lesson 8 using the FundMe contract of lesson 4, if it works then I'll style and optimized it.

[alymurtazamemon]

@pacelliv see this discussion #2364