Lesson 8 : inpage.js:1 MetaMask - RPC Error: [ethjs-query]

[ameeetgaikwad]

When I'm calling fund function through my front-end ans sending eth to the contract, I'm having this error.

My index.js file

//in node.js ... require

//in front-end javascript you can't use require ... import

import { ethers } from "./ethers-5.6.esm.min.js";
import { abi, contractAddress } from "./constants.js";

const connectButton = document.getElementById("connectButton");
const fundButton = document.getElementById("fundButton");
connectButton.onclick = connect;
fundButton.onclick = fund;

async function connect() {
    if (typeof window.ethereum !== "undefined") {
        await window.ethereum.request({ method: "eth_requestAccounts" });
        connectButton.innerHTML = "Connected!";
    } else {
        connectButton.innerHTML = "Please install metamask   ";
    }
}

async function fund(ethAmount) {
    ethAmount = "0.1";
    console.log(`Funding with ${ethAmount}...`);
    if (typeof window.ethereum !== "undefined") {
        //provider /connection to the blockchain
        //signer /wallet /someone with some gas
        //contract that we are interacting with
        //ABI and address
        const provider = new ethers.providers.Web3Provider(window.ethereum);
        const signer = provider.getSigner();
        const contract = new ethers.Contract(contractAddress, abi, signer);
        try {
            const transactionResponse = await contract.fund({
                value: ethers.utils.parseEther(ethAmount),
            });
        } catch (error) {
            console.log(error);
        }
    }
}

[krakxn]

Going by sheer memory here, but pretty sure the chainId is 1337 for Metamask specifically.

EDIT: Found the problem (lies in ethAmount), use this instead: const ethAmount = document.getElementById("ethAmount").value

Note for the above: the ID in the html file should be ethAmount for the above to work, or whatever you see fit, but both the IDs should match

[ameeetgaikwad]

Yep! It worked.
But what was the problem?

[krakxn]

1. Incorrect chainId 2. ethAmount did not pull the placeholder text from HTML ID: ethAmount

Also, if it is solved please close for others to refer to.

[ameeetgaikwad]

No the chainId is correct, what is wrong with just directly writing this: ethAmount="0.1" ?

[krakxn]

1. You can, but you are directly initializing it a value without declaring it beforehand 2. It is hard-coded, it is best to rather use the placeholder text (i.e. whatever the front-end displays) for value