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

Fix/fix scroll call contract bug #591

Merged
merged 29 commits into from Jul 28, 2023

Conversation

imotai
Copy link
Contributor

@imotai imotai commented Jul 28, 2023


PR-Codex overview

This PR focuses on introducing a new boolean flag use_legacy_tx to the MetaStoreClient struct. It allows the client to use either the legacy or the new transaction format.

Detailed summary

  • Added use_legacy_tx field to MetaStoreClient struct
  • Updated MetaStoreClient::new function to accept use_legacy_tx parameter
  • Modified relevant functions in MetaStoreClient to handle the new flag
  • Updated code in node_test_base.rs, recover.rs, command.rs, and rollup_executor.rs to use the use_legacy_tx flag

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@imotai imotai merged commit 9764d63 into dbpunk-labs:main Jul 28, 2023
4 checks passed
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.

None yet

1 participant