Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[don't pull]add deployer with enhancedStepsReserve #1050

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ducquangkstn
Copy link
Contributor

No description provided.

@ducquangkstn ducquangkstn force-pushed the addEnhancedOpenData-deployment branch from 6d3689f to 9f9b604 Compare July 31, 2020 07:57
Copy link
Contributor

@ilanDoron ilanDoron left a comment

Choose a reason for hiding this comment

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

nice

the deploy script is short and elegant.
always nice to have shorter code.

is nonce handled automatically?


async function main() {
const accounts = await web3.eth.getAccounts();
deployer = accounts[0];
Copy link
Contributor

Choose a reason for hiding this comment

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

plan is to send funds to this address?
if so how to return back remaining funds when done?
have private key?
consider saving it locally.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

if you set the network is ropsten in the script, accounts[0] will be the address from the privatekey in .env file

.catch(error => {
console.error(error);
process.exit(1);
});
Copy link
Contributor

Choose a reason for hiding this comment

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

new line

deployer = accounts[0];
console.log(`deployer address at ${deployer}`);

gasPrice = new BN(75).mul(new BN(10).pow(new BN(9)))
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe use yargs for gas price?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

if we run the script with buidler, we can not read value from args
but we can read the value from environment variable

@ducquangkstn ducquangkstn force-pushed the addEnhancedOpenData-deployment branch from 9f9b604 to 1fd44a3 Compare August 6, 2020 09:12
@ducquangkstn ducquangkstn changed the base branch from ilan/addEnhancedOpenData to master August 6, 2020 09:12
@ducquangkstn ducquangkstn force-pushed the addEnhancedOpenData-deployment branch 3 times, most recently from 1712d1d to 9006655 Compare August 6, 2020 11:26
@ducquangkstn ducquangkstn force-pushed the addEnhancedOpenData-deployment branch from 9006655 to 454fb4a Compare September 10, 2020 10:38
@ducquangkstn ducquangkstn force-pushed the addEnhancedOpenData-deployment branch from 454fb4a to 292ca30 Compare September 10, 2020 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants