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(contract cmds): Add send, call, deploy #67

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

fborello-lambda
Copy link
Contributor

@fborello-lambda fborello-lambda commented Sep 10, 2024

Purpose

  • Add basic contract interaction
  • Proposed command: zks contract ...

Example

❯ zks contract deploy --help
Deploy a contract.
        
Use zksolc to compile a contract, `zksolc ContractName.sol -O3 --combined-json abi` will give the combined json.
        
Pass contract_path parameter the combined.json file's path.
        
You can use --output-dir to compile and create a file automatically.
    

Usage: zks contract deploy [OPTIONS]

Options:
  -b, --bytecode <BYTECODE>                    Used for L1 deploy
  -p, --contract_path <COMBINED_JSON_PATH>     Combined json's path, generated by zksolc
  -a, --constructor_args <CONSTRUCTOR_ARGS>    
  -t, --constructor_types <CONSTRUCTOR_TYPES>  Used for L1 deploy
      --l1                                     
  -h, --help                                   Print help

Deploying an ERC20

zks contract deploy -p contract_abi_examples/StorageFibonacci.json    

@fborello-lambda fborello-lambda self-assigned this Sep 10, 2024
@fborello-lambda
Copy link
Contributor Author

We need to implement a way to deploy contracts with the new sdk

@fborello-lambda fborello-lambda marked this pull request as ready for review September 18, 2024 19:15
@fborello-lambda fborello-lambda marked this pull request as draft September 18, 2024 19:16
@fborello-lambda fborello-lambda marked this pull request as ready for review September 20, 2024 20:33
@fborello-lambda fborello-lambda removed the request for review from IAvecilla September 20, 2024 20:33
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.

1 participant