Lesson-4 Unable to send fund to contract via metamask - Failed transaction - out of gas

[Leonardll]

Hi guys, I am having issues sending funds on the contract. I am having error message-"Transaction failed". The etherscan details is showing ouf of gas as per screenshot attached.

If have also tried to increase the gas limit on metamask as advised but it does not work. Please see sreenshot below.

Here is my code
`// SPDX-License-Identifier: MIT
pragma solidity ^0.8.8;

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

error NotOwner();

contract FundMe {
using PriceConverter for uint256;

uint256 public constant MINIMUM_USD = 50 * 10**18;

address[] public funders;
mapping(address => uint256) public addressToAmountFunded;

address public immutable i_owner;

constructor () {
    i_owner = msg.sender;
}

function fund() public payable {
    // want to be able to set a minimum fund amount in 
    
    // 1 How do send ETH to this contract ?
    require(msg.value.getConversionRate() >= MINIMUM_USD, "Didn't send enough"); //1e18 == 1 * 10 ** 18 == 10000000000000000
    addressToAmountFunded[msg.sender] += msg.value;
    funders.push(msg.sender);
}


function withdraw() public onlyOwner {
    /* starting indedx, ending index, step amount*/
    for (uint256 funderIndex = 0; funderIndex < funders.length; funderIndex++) {
        funders[funderIndex];
        address funder = funders[funderIndex];
        addressToAmountFunded[funder] = 0;
    }
    // reset array
    funders = new address[](0);

    // actually withdraw the funds

    // transfer  --2300 gas throw err 
    //  payable (msg.sender).transfer(address(this).balance);
    
    //  // send
    //  bool sendSuccess =  payable(msg.sender).send(address(this).balance);
    //  require(sendSuccess, "send failed");
    // call
    (bool callSuccess,) = payable(msg.sender).call{value: address(this).balance}("");
    require(callSuccess, "Call failed");
    revert();


}
   
 modifier onlyOwner {
    // require(msg.sender == i_owner, "Sender is not owner!")
    if (msg.sender != i_owner) { revert NotOwner();}
    _;
}

// what happens if someone sends this contract ETH without calling the fund fun

// reveive and fallback 

receive() external payable {
    fund();
}

fallback() external payable {
    fund();
}

}
`

Can someone help ,please?

[shashikant-burnwal]

Please do these 2 things and see if it gets fixed:

1. On Remix IDE itself increase the gas limit and please increase it by at least 1000 x
2. On Metamask also please do the same

[Leonardll]

thanks for this, should i redeploy the contract with this new gas limit as well?

[shashikant-burnwal]

Yes, once you choose the new gas limit, please remove any existing deployed contract instance and again redeploy.

[Leonardll]

sending fund still fails but with a new error message on etherscan. It says cancelled. Most likely as a result of revert opcode.

gas limit in remix is 3000000000 and in metamask i put the max i could.

[DorianDaSilva]

@Leonardll Do you have enough Rinkeby ETH? or are you using the correct network?

[Leonardll]

Hi Dorian, yes i have enough, and i am on the right network

[alymurtazamemon]

@Leonardll

Please check with this price 40000000000000000. And make sure you are on Rinkeby and injected web3 is selected in the deploy and run transactions tab.

One more thing, it looks like you have not moved the AggregatorV3Interface import to the PriceConverter library. Please make sure it, because otherwise, you will not get the correct results.

[alymurtazamemon]

please show me your PriceConverter file content here.

[Leonardll]

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

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

library PriceConverter {
    
    
function getPrice() internal view returns (uint256) {
    //ABI 
    // Address 0x5f4eC3Df9cbd43714FE2740f5E3616155c5b8419
    AggregatorV3Interface priceFeed = AggregatorV3Interface(0x5f4eC3Df9cbd43714FE2740f5E3616155c5b8419);
   (,int256 price,,,) = priceFeed.latestRoundData();
   // ETH in USD
   // 3000.00000000
   return uint256(price * 1 ** 10); // 1 ** 10 == 10000000000

}

function getVersion() internal view returns (uint256) {
    AggregatorV3Interface priceFeed = AggregatorV3Interface(0x5f4eC3Df9cbd43714FE2740f5E3616155c5b8419);
    return priceFeed.version();
}

function getConversionRate(uint256 ethAmount) internal view returns (uint256) {
    uint256 ethPrice = getPrice();
    uint256 ethAmountInUsd = (ethPrice * ethAmount) / 1 ** 18;

    return ethAmountInUsd;

 } 


}

[alymurtazamemon]

here the address is not correct : 0x8A753747A1Fa494EC906cE90E9f37563A8AF630e

[Leonardll]

this is commented. i removed the comment to avoid confusion, sorry

[alymurtazamemon]

AggregatorV3Interface(0x5f4eC3Df9cbd43714FE2740f5E3616155c5b8419);

Change this address with 0x8A753747A1Fa494EC906cE90E9f37563A8AF630e for ETH/USD AggregatorV3Interface address is this.