-
|
I am trying to deploy Raffle to test if the script is working correctly (15:19:46 of the video) I keep getting this error I had a simliar problem when trying to deploy a mock for chapter 7 and the issue was in the named accounts in hardhat.config . I tried using that solution, as well as copying the deploy01 and hardhat.config from the repo and I'm still getting the same issue. |
Beta Was this translation helpful? Give feedback.
Replies: 5 comments 29 replies
-
|
Can you share the code? |
Beta Was this translation helpful? Give feedback.
-
|
here is 01-deploy-raffle.js const { network, ethers } = require("hardhat")
const { developmentChains, networkConfig } = require("../helper-hardhat-config")
const { verify } = require("../helper-hardhat-config")
// const VRD_SUB_FUND_AMOUNT = ethers.utils.parseEther("30")
const FUND_AMOUNT = "1000000000000000000000"
module.exports = async function ({ getNamedAccounts, deployments }) {
const { deploy, log } = deployments
const { deployer } = await getNamedAccounts()
const chainId = network.config.chainId
let vrfCoordinatorV2Address, subscriptionId
if (chainId == 31337) {
// create VRFV2 Subscription
const vrfCoordinatorV2Mock = await ethers.getContract("VRFCoordinatorV2Mock")
vrfCoordinatorV2Address = vrfCoordinatorV2Mock.address
const transactionResponse = await vrfCoordinatorV2Mock.createSubscription()
const transactionReceipt = await transactionResponse.wait()
subscriptionId = transactionReceipt.events[0].args.subId
// Fund the subscription
// Our mock makes it so we don't actually have to worry about sending fund
await vrfCoordinatorV2Mock.fundSubscription(subscriptionId, FUND_AMOUNT)
} else {
vrfCoordinatorV2Address = networkConfig[chainId]["vrfCoordinatorV2"]
subscriptionId = networkConfig[chainId]["subscriptionId"]
}
const entranceFee = networkConfig[chainId]["entranceFee"]
const gasLane = networkConfig[chainId]["gasLane"]
const callbackGasLimit = networkConfig[chainId]["callbackGas"]
const interval = networkConfig[chainId]["interval"]
const args = [
vrfCoordinatorV2Address,
entranceFee,
gasLane,
subscriptionId,
callbackGasLimit,
interval,
]
const raffle = await deploy("Raffle", {
from: deployer,
args: args,
log: true,
waitConfirmations: network.config.blockConfirmations || 1,
})
if (!developmentChains.includes(network.name) && process.env.ETHERSCAN_API_KEY) {
log("Verifying...")
await verify(raffle.address, args)
}
log("-------------------------------------------------")
}
module.exports.tags = ["all", "raffle"]and here is hardhat.config.js require("@nomiclabs/hardhat-waffle")
require("@nomiclabs/hardhat-etherscan")
require("hardhat-deploy")
require("solidity-coverage")
require("hardhat-gas-reporter")
require("hardhat-contract-sizer")
require("dotenv").config()
const RINKEBY_RPC_URL = process.env.RINKEBY_RPC_URL
const PRIVATE_KEY = process.env.PRIVATE_KEY
const COINMARKETCAP_API_KEY = process.env.COINMARKETCAP_API_KEY
const ETHERSCAN_API_KEY = process.env.ETHERSCAN_API_KEY
module.exports = {
defaultNetwork: "hardhat",
networks: {
hardhat: {
chainId: 31337,
blockConfirmations: 1,
},
rinkeby: {
chainId: 4,
blockConfirmattions: 6,
url: RINKEBY_RPC_URL,
accounts: [PRIVATE_KEY],
},
},
solidity: {
compilers: [
{
version: "0.8.7",
},
{
version: "0.4.24",
},
],
},
namedAccounts: {
deployer: {
default: 0, // here this will by default take the first account as deployer
1: 0, // similarly on mainnet it will take the first account as deployer. Note though that depending on how hardhat network are configured, the account 0 on one network can be different than on another
},
player: {
default: 1,
},
},
} |
Beta Was this translation helpful? Give feedback.
-
|
One error I can spot right now is missing no of blocks to wait . And please share your solidity file too. |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for youre help on this. Unfortunatly passing 1 for no of blocks to wait didnt work. Here is my solidity file: |
Beta Was this translation helpful? Give feedback.
-
|
Found another error here we are looking for |
Beta Was this translation helpful? Give feedback.
-
|
Thanks but unfortunatly this didnt solve the issue. Here is my code as of now: Error: ERROR processing /Users/carson/repos/freeCodeCamp_lessons/full-blockchain-solidity-course-js/hardhat-smartcontract-lottery-fcc/deploy/01-deploy-raffle.js: |
Beta Was this translation helpful? Give feedback.
-
|
@cah311 Push your code to GitHub and leave your repo link here. |
Beta Was this translation helpful? Give feedback.
-
|
https://github.com/cah311/hardhat-smartcontract-lottery-fcc Thanks for checking this out! |
Beta Was this translation helpful? Give feedback.
-
|
@cah311 Delete your args variable and replace it with mine and let me know if it fixes your problem: const arguments = [
vrfCoordinatorV2Address,
subscriptionId,
networkConfig[chainId]["gasLane"],
networkConfig[chainId]["keepersUpdateInterval"],
networkConfig[chainId]["raffleEntranceFee"],
networkConfig[chainId]["callbackGasLimit"],
]Also, rename any other instances (in deploy and verify in the same file) to |
Beta Was this translation helpful? Give feedback.
-
|
hey thanks both of you for all your help on this. Unfortunatly I am still getting the same subid error. I pulled the changes from github and I added the same subId that is in the repo. I am not sure why this didnt change anything. |
Beta Was this translation helpful? Give feedback.
-
|
Did you create your own subId? If not, make one. |
Beta Was this translation helpful? Give feedback.
-
|
Sorry but I dont understand what you mean by creating a sub id? In the repo it has "588" there so that's what I tried. |
Beta Was this translation helpful? Give feedback.
-
|
All good, but please pay attention to the course; Patrick tells us to make one and where from. Follow this: Deployment to a testnet or mainnet
You'll want to set your
Head over to faucets.chain.link and get some tesnet ETH & LINK. You should see the ETH and LINK show up in your metamask. You can read more on setting up your wallet with LINK.
Head over to vrf.chain.link and setup a new subscription, and get a subscriptionId. You can reuse an old subscription if you already have one. You can follow the instructions if you get lost. You should leave this step with:
In your Then run: And copy / remember the contract address.
Go back to vrf.chain.link and under your subscription add
You can follow the documentation if you get lost. Go to keepers.chain.link and register a new upkeep. Your UI will look something like this once completed:
You're contract is now setup to be a tamper proof autonomous verifiably random lottery. Enter the lottery by running: Setup a Chainlink VRF Subscription ID You can follow the instructions if you get lost. You should leave this step with: A subscription ID Your subscription should be funded with LINK Deploy |
Beta Was this translation helpful? Give feedback.
-
|
Ok thanks sorry I knew how to do the subscription part. I did follow the tutorial so I should have understood. I actually went back and redid it because I thought I missed something and I didnt. For this part that I am stuck on (15:20 of the video), Patrick actually leaves the subId at "0" because he says he wants to show us how to create one locally later on. But for some reason his deploy passes and even after starting the chapter over I am still stuck at the same place. I even copied the updated repo again and created a new subID. I think this time the subId is passing, but now I am getting this error: |
Beta Was this translation helpful? Give feedback.
-
|
Print all the variables which you get from networkConfig object i.e.: console.log(`-------------1---------${entranceFeeToEnterTheLottery}------------------------------`);
console.log(`-------------2---------${gasLane}------------------------------`);
console.log(`-------------3---------${callBackGasLimit}------------------------------`);
console.log(`-------------4---------${lotteryTimeInterval}------------------------------`);
console.log(`-------------5---------${subscriptionId}------------------------------`);One of them will be undefiend either you misspelled the name incorrectly in helper-hardhat-config.js or in 01-deploy-raffle.js. |
Beta Was this translation helpful? Give feedback.
-
|
Where do I put this and in which file? I tried placing it a few places within 01-deploy-raffle.js as well as helper-hardhat-config.js but the console.logs never run because the error runs before them. |
Beta Was this translation helpful? Give feedback.
-
|
You have to put it in 01-deploy-raffle.js exactly after this lines } else {
vrfCoordinatorV2Address = networkConfig[chainId]["vrfCoordinatorV2"]
subscriptionId = networkConfig[chainId]["subscriptionId"]
}
const entranceFee = networkConfig[chainId]["entranceFee"]
const gasLane = networkConfig[chainId]["gasLane"]
const callbackGasLimit = networkConfig[chainId]["callbackGas"]
const interval = networkConfig[chainId]["interval"]Names of your variable might be different to try tweaking those. console.log(`-------------1---------${entranceFee}------------------------------`);
console.log(`-------------2---------${gasLane}------------------------------`);
console.log(`-------------3---------${callbackGasLimit}------------------------------`);
console.log(`-------------4---------${interval}------------------------------`);
console.log(`-------------5---------${subscriptionId}------------------------------`);like this. |
Beta Was this translation helpful? Give feedback.
-
|
Thanks again. This helped. I was able to fix the mistakes but now I am getting this error: |
Beta Was this translation helpful? Give feedback.
-
|
In helper-hardhat-config-js file entranceFee: ethers.utils.parseEther("0.01"),If your problem is still not solved make a github repo so that we can directly look into the problem there. |
Beta Was this translation helpful? Give feedback.
-
|
Yeah entranceFee did have that value. Here is my repo: |
Beta Was this translation helpful? Give feedback.
-
|
Thank you everyone for trying to help me out. Unfortunatly with each piece of advice, a new error has come up. As of now I feel like I am very close to the solution, but I am stuck on this issue now: const { getNamedAccounts, deployments, network, run } = require("hardhat") const FUND_AMOUNT = "1000000000000000000000" module.exports = async ({ getNamedAccounts, deployments }) => { } module.exports.tags = ["all", "raffle"] `` |
Beta Was this translation helpful? Give feedback.
-
|
Hey the sequence of arguments in the contract and the deploy script are different...that might be the cause of error |
Beta Was this translation helpful? Give feedback.
-
|
Just sent a pull request with the correct order, |
Beta Was this translation helpful? Give feedback.
-
|
thank you so much this finally worked. |
Beta Was this translation helpful? Give feedback.
-
|
Glad that I could help🙂. |
Beta Was this translation helpful? Give feedback.
Thank you everyone for trying to help me out. Unfortunatly with each piece of advice, a new error has come up. As of now I feel like I am very close to the solution, but I am stuck on this issue now: