General Thread for practicing question formatting: Lession 5!

[akoffikumassi]

function fund() public payable {
        require(msg.value.getConversionRate() >= MINIMUM_USD, "You need to spend more ETH!");
        // require(PriceConverter.getConversionRate(msg.value) >= MINIMUM_USD, "You need to spend more ETH!");
        addressToAmountFunded[msg.sender] += msg.value;
        funders.push(msg.sender);
    }
    

TypeError: "msg.value" and "callvalue()" can only be used in payable public functions. Make the function "payable" or use an internal function to avoid this error

Can someoe tell me what's going on

[SantanuDK]

Have you set funders array and addressToAmountFunded mapping as public ?

[alwayscommit]

I think the code that you have shared does not have any issues, could you share the entire solidity file?

Also, even if the "addressToAmountFunded " does not have "public" visibility, the fund() function still works because "addressToAmountFunded " is treated as internal, which basically means that you won't be able to lookup the funded amount by different addresses from the map but the fund functionality still works fine.

[PatrickAlphaC]

Can you share your library and getConversionRate function?