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

feat: cast send --create #2871

Merged
merged 10 commits into from Aug 22, 2022
Merged

feat: cast send --create #2871

merged 10 commits into from Aug 22, 2022

Conversation

gakonst
Copy link
Member

@gakonst gakonst commented Aug 22, 2022

Previously, cast send would not allow for an optional to, which would preclude people from deploying raw bytecode. This addresses it in the same way that cast estimate --create works. Also refactors the cast send logic to a separate file.

@stong
Copy link

stong commented Aug 22, 2022

Works like a charm! Thank you @gakonst

Copy link
Collaborator

@rkrasiuk rkrasiuk left a comment

lgtm

} // Checking if signer isn't the default value
// 00a329c0648769A73afAc7F9381E08FB43dBEA72.
} else if config.sender !=
Address::from_str("00a329c0648769A73afAc7F9381E08FB43dBEA72").unwrap()
Copy link
Collaborator

@rkrasiuk rkrasiuk Aug 22, 2022

Choose a reason for hiding this comment

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

shall we move this to a constant ?

Copy link
Collaborator

@joshieDo joshieDo Aug 22, 2022

Choose a reason for hiding this comment

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

Config::DEFAULT_SENDER

Copy link
Member

@mattsse mattsse Aug 22, 2022

Choose a reason for hiding this comment

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

yeah good idea,
can do this in a followup since this is used in a bunch of places

cli/src/cmd/cast/send.rs Outdated Show resolved Hide resolved
mattsse and others added 2 commits Aug 22, 2022
@mattsse mattsse merged commit 3dec598 into master Aug 22, 2022
15 checks passed
@mattsse mattsse deleted the cast-send-create branch Aug 22, 2022
@gakonst
Copy link
Member Author

gakonst commented Aug 22, 2022

thank you sirs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cast T-feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants