Remix Fund Me - false Transaction mined but execution failed

[LordReya]

Always getting this error in this section. What should I do?
I send enough eth for this transaction...

status	false Transaction mined but execution failed
transaction hash	0xb7b8b6f2218f83696ff9df3fd2a1da8ed5eca55c7e190030086b7e9678ae2be1
from	0xc2f99d7A9A103978A488F8D170CFa0B42990D2c6
to	FundMe.(constructor)
gas	3000000 gas
transaction cost	103407 gas
input	0x608...70033
decoded input	{}
decoded output	-
logs	[]
val	30907900000000000 wei

This is my code:

// get funds, withdraw funds, set minimum funding value to USD

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

import "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol"; //Import from Github directly (better than copy past

contract FundMe {

uint256 public minimumUsd = 50 * 1e18;

    function fund() public payable{ 
//1. how to send etherium to account?
// set minimum usd sending value 
require(getConversionRate(msg.value) >= minimumUsd, "Didn't send enough");
//1e18 == 1*10 ** 18 == 100000000000000000

    }

function getPrice() public view returns(uint256){
    //ABI
    //Address 0x8A753747A1Fa494EC906cE90E9f37563A8AF630e
        AggregatorV3Interface priceFeed = AggregatorV3Interface(0x8A753747A1Fa494EC906cE90E9f37563A8AF630e);
        (,int256 price,,,) = priceFeed.latestRoundData();
        return uint256(price * 1e10); //1**10 = 10000000000
    }

    function getVersion() public view returns(uint256) {
        AggregatorV3Interface priceFeed = AggregatorV3Interface(0x8A753747A1Fa494EC906cE90E9f37563A8AF630e);
        return priceFeed.version();
    }
    function getConversionRate(uint256 ethAmount) public view returns(uint256) {
        uint256 ethPrice = getPrice();
        uint256 ethAmountInUsd = (ethPrice * ethAmount) / 1e18;
        return ethAmountInUsd;
    }

    //function withdraw(){}
}
//payable function needed to send money, require and msg.value for minimum sending value

[dibakarsutradhar]

try setting the minimumUSD to 50 * 10**18

uint256 minimumUSD = 50 * 10**18;

[dibakarsutradhar]

My bad, I kinda missed the error. The problem is with your constructor. The transaction is looking for a to address, but the contract is returning `FundMe.(constructor).

You should set up a constructor like this -

address owner;

    constructor() public {
        owner = msg.sender;
    }

You also need a new public variable funders and mapping addressToAmountFunded and call them in the fund function.
Something like this -

mapping(address => uint256) public addressToAmountFunded;

address[] public funders;
address owner;

constructor() public {
    owner = msg.sender;
}

function fund() public payable {
    uint256 minimumUSD = 50 * 10**18;
    require(
        getConverstionRate(msg.value) >= minimumUSD,
        "You need to spend more ETH!"
    );
    addressToAmountFunded[msg.sender] += msg.value;
    funders.push(msg.sender);
}

[LordReya]

It worked, thank you!

[TilakMaddy]

Hey ... you could do this for highlighting the code

```solidity
---- code here --
```

[LordReya]

Hey ... you could do this for highlighting the code

solidity ---- code here --

Just went through this section in the course and updated my code up there. Thank you for the advice :)

[PatrickAlphaC]

Thanks @TilakMaddy !

Great update @LordReya !

[Suzie1899]

Hi, I had the same error and I tried this solution but now I am getting another error saying my 'getConversionRate' in the require function is an undeclared identifier. I have imported the PriceConverter library. How do I fix this?

[PatrickAlphaC]

Can you ask this on stack exchange ETH?