Lesson 4:

[abdullahtanvir11]

This error occurs when I click on funds

"Gas estimation errored with the following message (see below). The transaction execution will likely fail. Do you want to force sending?
execution reverted"

[RoboCrypter]

@abdullahtanvir11 : You should deploy your contract on goerli

If you are deploying on rinkeby as I think, It should be the cause of this error!

[RoboCrypter]

@abdullahtanvir11 : Give me a moment!

[RoboCrypter]

@abdullahtanvir11 : Copy & Paste this one, I'm giving you below I have edited it, and deploy your contract again!

// SPDX-License-Identifier: MIT
pragma solidity ^0.8.8;

import "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol";

// Why is this a library and not abstract?
// Why not an interface?
library PriceConverter {
// We could make this public, but then we'd have to deploy it
function getPrice() internal view returns (uint256) {
// Goerli ETH / USD Address
// https://docs.chain.link/docs/ethereum-addresses/
AggregatorV3Interface priceFeed = AggregatorV3Interface(
0xD4a33860578De61DBAbDc8BFdb98FD742fA7028e
);
(, int256 answer, , , ) = priceFeed.latestRoundData();
// ETH/USD rate in 18 digit
return uint256(answer * 10000000000);
}

// 1000000000
function getConversionRate(uint256 ethAmount)
    internal
    view
    returns (uint256)
{
    uint256 ethPrice = getPrice();
    uint256 ethAmountInUsd = (ethPrice * ethAmount) / 1000000000000000000;
    // the actual ETH/USD conversion rate, after adjusting the extra 0s.
    return ethAmountInUsd;
}
}

Look at that 👍

It will work for sure!

[abdullahtanvir11]

Yes it worked... Can you tell what have you changed? @abossofmyself

[RoboCrypter]

@abdullahtanvir11 : Awesome!

Price feed was wrong you have to use goerli price feed in order for it to work, It wasn't goerli's!

It just put the goerli price feed and removed the previous one and it worked!

Aka this one : 0xD4a33860578De61DBAbDc8BFdb98FD742fA7028e

[abdullahtanvir11]

Thankyou bro... :)

[krakxn]

The code snippet please.

[krakxn]

This

addressToAmountFunded[msg.sender] = msg.value;

should be

addressToAmountFunded[msg.sender] += msg.value;

instead (+=).

So,

-addressToAmountFunded[msg.sender] = msg.value;
+addressToAmountFunded[msg.sender] += msg.value;

[abdullahtanvir11]

still not working

[krakxn]

still not working

@abdullahtanvir11 It is an issue on your end. The logic of the code is correct; I used the same code.

Make sure your MetaMask is on the right network and with sufficient funds (make sure the testnet is Goerli as well, and the subsequent Goerli Ether and not some other token, also make sure of the network in Remix)

Another reason would be not deploying the contract after the changes; therefore, it is a possibility you are running the fund function of the old contract.

[abdullahtanvir11]

@krakxn can you tell me amount in wei and i have 2.4 in my goerli account

[krakxn]

@krakxn can you tell me amount in wei and i have 2.4 in my goerli account

That is sufficient for the txn.

Try the other reasons.

Additionally, after doing the above, make sure you are on Injected Web3 as well as resetting your MetaMask account.

I have probably listed all the possibilities and once you consider them all, the issue should be fixed.

[alymurtazamemon]

@abdullahtanvir11 Make sure to do changes that above they have mentioned and another thing ETH/USD price feed address is this 0xD4a33860578De61DBAbDc8BFdb98FD742fA7028e, you are using ETH/USD mainnet address. Add this to the Aggregator interface address.