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

[New Feature] Optional CLI Argument for ProveTxHash #36

Open
2 tasks
koloz193 opened this issue Mar 5, 2024 · 0 comments
Open
2 tasks

[New Feature] Optional CLI Argument for ProveTxHash #36

koloz193 opened this issue Mar 5, 2024 · 0 comments

Comments

@koloz193
Copy link
Contributor

koloz193 commented Mar 5, 2024

Motivation

In the current system we use the L2 rpc to grab the commitBatch transaction hash and the proveBatch tx hash. There are some scenarios where the L1 will get this proveBatches transaction before it is reflected in the RPC response. We want to add an optional cli argument for the proveTxHash that will supersede what's returned in that call.

Acceptance Criteria

  • Add new proveTxHash cli argument
  • Update function to use the proveTxHash when supplied, properly handling when both are empty

References

This pr added a new cli argument for the schedule key file path

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

No branches or pull requests

1 participant