Lesson 4: Receive & Fall back

[SummerShyne]

Hello, I need help with when I send eth to my contract via Metamask, I get an error in rinkeby etherscan saying " Warning! Error encountered during contract execution [out of gas] "

This is for the Fundme.sol file

I have wrote the code below

receive() external payable{
fund();
}

fallback() external payable{
    fund(); 

}

These were the lines that Patrick showed during the video where doing that should let me send eth to the contract via MetaMask, In solidity the code compiled correctly and everything is fine but I am not able to fund the contract with Eth via metamask but I am able to fund it via the contract meaning via Remix. any help would be appreciated. Thanks again

[mattjaf]

what does your fund() function look like

might of not met a require statement

[SummerShyne]

function fund() public payable {
require(msg.value.getConversionRate() >= MINIMUM_USD, "Didn't send enough!");
funders.push(msg.sender);
addressToAmountFunded[msg.sender] += msg.value;

[mattjaf]

It seems like your not meeting the require(msg.value.getConversionRate() >= MINIMUM_USD, "Didn't send enough!");

how much did you try to send it?

At the current price of 1,053, you gotta send at least .05 eth at the time of writing this, which is 52.66$ in USD testnet.

[EngAbdalrhman]

I sent 0.02 eth and by fund function, it work with only 0.15 eth

[EngAbdalrhman]

just didn't notice that i was trying in another version of deployed contract

[alymurtazamemon]

@SummerShyne Make sure you have enough funds in your wallet account and pass value according to the correct price as @mattjaf mentioned.

After doing this if you still face the issue then you can manually set the gas cost when the MetaMask will popup. By default, the base fee is set the 21000, if computation costs more gas then we need to give more gas.

Wallets have the feature to automatically detect the gas cost, if MetaMask is failing in doing so, then should manually select the higher fee, and all the remaining gas will return to the wallet.

[TotalingArc]

im having the same issue, but I have tried different amounts, not working. My code is the same as video.

`// Get funds from users
// Withdraw funds
// Set a minimum funding value in USD

//SPDX-License-Identifier: MIT

pragma solidity ^0.8.8;

import "./PriceConverter.sol";

error NotOwner();

contract FundMe{

using  PriceConverter for uint256;

uint256 public constant MINIMUM_USD = 25 * 1e18; // 1 * 10 ** 18

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

address public immutable i_owner;

constructor(){
    i_owner = msg.sender;
}

function fund() public payable{
    //Set minimum fund in USD
    //How do we send ETH to this contract
    require(msg.value.getConversionRate() >= MINIMUM_USD, "Not enough Ether sent"); //1e18 = 1 * 10 ** 18 == 1000000000000000000
    funders.push(msg.sender);
    addressToAmountFunded[msg.sender] += msg.value;


    //PAYABLE allows contracts to receive ether
    //REVERT
    //undo any action before, and send remaining gas back
}



 function withdraw() public onlyOwner {

 
    /*START, END, STEP */
    for(uint256 funderIndex = 0; funderIndex < funders.length; funderIndex++){

        address funder = funders[funderIndex];
        addressToAmountFunded[funder] = 0;
    }

    //reset array
    funders = new address[](0);
   
    // withdraw funds

    /*//transfer
    payable(msg.sender).transfer(address(this).balance);
    */

    /*//send
    bool sendSuccess = payable(msg.sender).send(address(this).balance);
    require(sendSuccess, "Send Failed");
    */


    //call - lower level, allows return on varialble data
    //Calls any function in ETH

    (bool callSuccess,) = payable(msg.sender).call{value: address(this).balance}("");
    require(callSuccess, "Call failed");

  

 }

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

    receive() external payable{
        fund();
    }

    fallback() external payable{
        fund();
    }

}`

[mattjaf]

i breifly reviewed this, you might have to send 1 extra wei due to truncation