Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

Update deploy.js #109

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update deploy.js #109

wants to merge 1 commit into from

Conversation

semz-ui
Copy link

@semz-ui semz-ui commented Dec 28, 2023

Update on how to get contract address anytime a transaction is being made

Update on how to get contract address anytime a transaction is being made
@semz-ui
Copy link
Author

semz-ui commented Dec 28, 2023

Note: This is an update for etherjs 6.0^. Also, I commented out the old method on how to get the address right above the new method

@PatrickAlphaC
Copy link
Owner

Thanks. We'd need to update all dependencies as well for this... Not sure this is enough in it's current state.

Copy link

@Thafoxes Thafoxes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

contract.address will get undefined, newer version contract.getAddress() does work but making it so it will be same as outputting deploymentReceipt.contractAddress . I'm not sure is it suppose to be same?

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants