Problem with Fallback/Receive functions.

[JeremySolidityMage]

Hi, does anyone know why when i deploy the contract on mainnet, when I send eth to contract through metamask, I do not have any balance assigned to my address and I cannot withdraw. I would be amazed if someone can help.

// SPDX-License-Identifier: No-License
pragma solidity 0.8.8;
import "./PriceConverter.sol";


error NotAuthorized();

contract FundMe {
    using PriceConverter for uint256;

    mapping(address => uint256) balances;
    uint256 public constant MINIMUMTHRESHOLDINUSD = 180 * 1e18; //180$$$ minimum to deposit
                                                         // CONSTANT means it cannot be changed later on. we will SAVE GAS *GAS OPTIMIZATION*

    address public immutable i_owner; // when IMMUTABLE. it will SAVE GAS. *GAS OPTIMIZATION*
    
    constructor() {
        i_owner = msg.sender;
    }
    function deposit() public payable {
        require(msg.value.getConversionRate() >= MINIMUMTHRESHOLDINUSD, "did not send enough"); // 1e18 = 1 ether
        balances[msg.sender] += msg.value;
    }

    function withdraw(uint256 _valueToWithdraw) public payable {
        require(balances[msg.sender] >= _valueToWithdraw);
        balances[msg.sender] -= (_valueToWithdraw);
        (bool Success, ) = payable(msg.sender).call{value: _valueToWithdraw}("");
        require(Success, "not successful, reverting...");
    }

    function GetMoney(address _addy) public view returns(uint256){
        return balances[_addy] / 1e18;
    }

    modifier OnlyThisAddressCan { //this modifier isnt used anywhere yet
        if (msg.sender != i_owner) { revert NotAuthorized(); } //SAVING MORE GAS than require(msg.sender == owner); *GAS OPTIMIZATION*
        _;
    }

    receive() external payable {
        deposit();
    }

    fallback() external  payable {
        deposit();
    }
}

[L1xus]

did you tried it on testnet first ?
does your code runs correctly without errors ?!

[JeremySolidityMage]

yes it runs without errors. and i test only on testnet since i cannot interact with Chainlink Oracle contracts on js evm environment

[alymurtazamemon]

@JeremySolidityMage You have defined the deposit function inside your fallback and receive functions, and it contains a require statement where you need at least $180 (I hope you are not doing this on mainnet 🤯) and then it compares with the conversion price, so I hope you have used the correct mainnet address for price conversion.