Lesson 4 - Fund Me not working - Transaction reverted

[Leonardll]

contract FundMe {
    using PriceConverter for uint256;

    uint256 public minimumUsd = 50 * 10**18;

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

    address public owner;

    constructor () {
        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() >= minimumUsd, "Didn't send enough"); //1e18 == 1 * 10 ** 18 == 10000000000000000
        addressToAmountFunded[msg.sender] += msg.value;
        funders.push(msg.sender);
    }

    

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

Hi guys, i am unable to fund the wallet, transaction get reverted automatically. Can someone help, please?

[alwayscommit]

You can try out a few things here

1. Comment out the require check which will narrow down the error to the conversionRate() function, if the transaction is successful with any amount.
2. If the transaction is successful without the require check, then you look into your conversion logic, and verify whether you're sending the correct amount of wei, over 50 dollars based on the current market price.
3. Check if you're deploying the contract on the correct network, on which the Chainlink priceFeed is active.

[PatrickAlphaC]

Great suggestion

[Leonardll]

thanks for this, it was related to the converter. It seems that 1e10 or 1e18 have to written 1 ** 10 and 1 ** 18.

[Equious]

When it's reverted, does it throw your exception of "Didn't send enough"? .. Can you post the error, please?

[PatrickAlphaC]

Great question

[Leonardll]

thanks for trying to help :)

[Equious]

Welcome Fren!