-
|
BrownieFundMeProject is the active project. Launching 'ganache-cli --port 8545 --gasLimit 12000000 --accounts 10 --hardfork istanbul --mnemonic brownie'... Running 'scripts/deploy.py::main'... Mocks deployed! Invalid command. Did you mean 'brownie compile'? Project has been compiled. Build artifacts saved at /Users/mattwilson/demos/brownie_fund_me/build/contracts BrownieFundMeProject is the active project. Launching 'ganache-cli --port 8545 --gasLimit 12000000 --accounts 10 --hardfork istanbul --mnemonic brownie'... Running 'scripts/deploy.py::main'... Mocks deployed! i am currently getting this error on lesson 6 when i try to deploy the project. i have the constructor as this atm... constructor() public { because i tried this one... constructor(address _priceFeed) public { i deleted build folder and when i tried to compile again it said "priceFeed" undeclared identifier. can i please get some help with this issue i am not too sure what to do. also i am quite new to coding and i find it really difficult of where to start when trying to solve an issue. is this a way to become better at identifying the source of a problem? |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 5 replies
-
|
Hello @mattwilsonbmx this error should be happening on your deploy script, as you said the constructor requires no arguments, so when you use deploy are you sending anything else rather than the account? could you share your deploy script plz. |
Beta Was this translation helpful? Give feedback.
-
|
from brownie import network, config, FundMe, MockV3Aggregator from scripts.helpful_scripts import get_account def deploy_fund_me():
account = get_account()
if network.show_active() != "development":
price_feed_address = config["networks"][network.show_active()][
"eth_usd_price_feed"
]
else:
print(f"The active network is {network.show_active()}")
print("Deploying mocks...")
mock_aggregator = MockV3Aggregator.deploy(
18, 2000000000000000000000, {"from": account}
)
price_feed_address = mock_aggregator.address
print("Mocks deployed!")
fund_me = FundMe.deploy(
price_feed_address,
{"from": account},
publish_source=config["networks"][network.show_active()].get("verify"),
)
print(f"Contract deploy to {fund_me.address}")
def main():
deploy_fund_me() |
Beta Was this translation helpful? Give feedback.
-
|
As you can see you are sending price_feed_address as parameter on: fund_me = FundMe.deploy(
price_feed_address,
{"from": account},
publish_source=config["networks"][network.show_active()].get("verify"),
)but your solidity code does not have it constructor(address _priceFeed) public {
priceFeed = AggregatorV3Interface(_priceFeed);
owner = msg.sender;
}
so you must send the pricefeed address or remove it from the deploy. |
Beta Was this translation helpful? Give feedback.
-
|
sorry bro thank you the help, but i dont know quite what you mean by this. do you mean, if i have this... constructor(address _priceFeed) public { in my constructor, i must change _priceFeed to price_feed_address? |
Beta Was this translation helpful? Give feedback.
-
|
i fixed it by removed the price_feed_address line in FundMe.deploy. Thanks for your help |
Beta Was this translation helpful? Give feedback.
-
|
That exactly what I meant, the price feed address was a constructor parameter, but you didn't have it declared on your function, I'm glad is solved now closing this thread,. |
Beta Was this translation helpful? Give feedback.
Hello @mattwilsonbmx this error should be happening on your deploy script, as you said the constructor requires no arguments, so when you use deploy are you sending anything else rather than the account? could you share your deploy script plz.