Lesson4 : getting unexpected error --> insufficient fund ,(instead of the expected) sender is not owner

[yeskaydee]

replicating the 5:03:28 / 1:07:54:30 , i am supposed to get this

but instead i am getting the (top most pic ) as result

//get funds ftom user
//Withdraw Funds
//Set a minimum funding value in usd
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.8;

import"./PriceConverter.sol";

contract FundMe{
    using PriceConverter for uint256;

    uint256 public minUSD = 50 * 1e18;


    //data structure to keep track of funders
    address[] public funders;

    //mapping structure to show how much money each donor has contributed
    mapping (address => uint256) public addressToAmountFunded;//is toh array ke tareh isko index value chaiye hota hai.

    // constructor(){
    //     minUSD=2;
    // }
    
    address public owner;

    constructor(){
        owner = msg.sender;
    }


    function fund() public  payable { // public so that everyone can fund it 
        //want to set a min  donate usd value
        //Q1.  how do we send eth to this contract?
        //Step-2 to acheieve the msg.value.getConversionRate();--copy to PriceConverter library
        //msg.value.getConversionRate();
        // require( getConversionRate(msg.value) > minUSD , "Not enough"); //1e18 is 1 eth
        
        // Final changes is 
        require(msg.value.getConversionRate() > minUSD , "Not enough"); //msg.value is considered as first parameter for the getConversionRate()
        

        //how to convert usd and eth interchangebly to you know compare them ? 
        //soln :- use decentralised oracle network

        //Whaat is reverting
        //undos any action before, and send remaining gas back


        //push funders to array -->funders<--
        funders.push(msg.sender);

        //kitna paise diye isne
        addressToAmountFunded[msg.sender] = msg.value;
    }

    

    function withdraw() public onlyOwner { //onlyOwner => do the onlyOwner first then the code
        // require(msg.sender==owner,"Sender is not owner"); --> don't want to copy require owner in every fn => modifier
        //for loop
        for(uint256 funderIndex=0; funderIndex < funders.length ; funderIndex++){
            //code
            address funder = funders[funderIndex]; //funders and funder is different, funders is array of adress, while funder is address of sender at that particular address
            addressToAmountFunded[funder]=0;

        }

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

        //actually withdraw the funds
        //3 diff ways - transfer, send, call

        //transfer
        // msg.sender is of type address
        // payable (msg.sender) is of type payable address
        payable (msg.sender).transfer(address(this).balance);


        //send-- if transcation fails , it wont send money back , to solve this we use bool
        bool sendSuccess=payable(msg.sender).send(address(this).balance);
        require(sendSuccess,"Send failed");

        //call -- inceredibly powerful, doesn't even need abi
        (bool callSuccess, ) = payable(msg.sender).call{value: address(this).balance}(""); //we only care about one variable
        require(callSuccess, "Call failed");

    }
modifier onlyOwner{
    require(msg.sender==owner,"Sender is not owner");
    _;  //means do rest of the code
}
    

}

sorry for the length of the code, it is mostly my notes in the form of comments, pls help me find the issue

[junaid5598]

Your code is very much fine. Actually your Metamask account doesn't have testnet eth (Sepolia ETH).
Go to https://faucets.chain.link/sepolia and get some Sepolia Testnet Faucets and try again. I'm pretty sure this should work.

[yeskaydee]

@junaid5598 thank you for the reply, but the issue is that i should get a notification that i am the invalid user, instead i am being notified that there is insufficient funds.

[alymurtazamemon]

@yeskaydee Click the Send Transaction and then it will show you the error.

[yeskaydee]

Ohh, will try