Practice

[hritisharma]

//SPDX-License-Identifier: MIT

pragma solidity ^0.8.8;

import "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol";
import "./PriceConverter.sol";

error NotOwner();

contract FundMe {

    using PriceConverter for uint256;

    uint256 public constant MINIMUM_USD =50 * 1e18;
    address[] public funders;
    mapping(address => uint256) public addressToAmount;

    address public immutable i_owner;

    constructor() {
        i_owner = msg.sender;
    }
  
    function fund() public payable {

        require(msg.value.getConversionRate() >= MINIMUM_USD, "Didn't send enough");
        funders.push(msg.sender);
        addressToAmount[msg.sender] = msg.value;
        
    }

    function withdraw() public onlyOwner {
        for(uint256 FunderIndex = 0; FunderIndex < funders.length; FunderIndex++)
        {
           address funder = funders[FunderIndex];
           addressToAmount[funder] = 0;
        }

        funders = new address[] (0);

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

    modifier onlyOwner {
        if(msg.sender != i_owner)
        {
            revert NotOwner();
        }
                _;
    }


    receive() external payable {
        fund();
    }

    fallback() external payable {
        fund();
    }


} 

On this function , I'm running into an error:

"msg.value" and "callvalue()" can only be used in payable public functions. 

Can someone tell me what's going on ?

[jayanand05]

Your code seems fine can you elaborate what error you are getting?

[OmarMohamed2001]

try adding (public) to the withdraw function, as it has callValue() and it requires to be a public function