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: zks namespace #2167

Merged
merged 19 commits into from
May 19, 2024
Merged

Conversation

nikola-bozin-txfusion
Copy link
Contributor

@nikola-bozin-txfusion nikola-bozin-txfusion commented Apr 22, 2024

Overview

This PR focuses on adding Public L2 actions functions for zkSync.

Detailed Summary:
Constructed zksRpcScheme for zks namespace.
Added new Actions which interact with zksRpcScheme methods.
Added unit tests for each Action.
Added PublicActionsL2 decorator and tests.


PR-Codex overview

This PR focuses on adding support for zks_ namespace and actions in the zkSync Extension.

Detailed summary

  • Added support for zks_ namespace and actions in zkSync Extension
  • Defined new types like MessageProof, BridgeContractAddresses, ZkSyncFeeValues
  • Implemented utility function camelCaseKeys
  • Added new environment variables in .env.example
  • Added new test files for various actions
  • Updated documentation for new actions: getL1ChainId, getL1BatchNumber, getBaseTokenL1Address, getMainContractAddress, getBridgehubContractAddress
  • Removed unused code in test/setup.ts

The following files were skipped due to too many changes: site/pages/zksync/actions/getBridgehubContractAddress.md, site/pages/zksync/actions/getTestnetPaymasterAddress.md, src/zksync/actions/getL1ChainId.ts, src/zksync/actions/getTransactionDetails.test.ts, src/zksync/actions/getL1BatchNumber.ts, src/zksync/actions/getBaseTokenL1Address.ts, src/zksync/actions/getMainContractAddress.ts, src/zksync/actions/getBridgehubContractAddress.ts, src/zksync/actions/getTestnetPaymasterAddress.ts, site/pages/zksync/actions/getDefaultBridgeAddress.md, site/pages/zksync/actions/getRawBlockTransactions.md, site/pages/zksync/actions/getL1BatchDetails.md, src/zksync/actions/getBlockDetails.test.ts, src/zksync/actions/getTestnetPaymasterAddress.test.ts, src/zksync/actions/estimateGasL1ToL2.test.ts, site/pages/zksync/actions/getBlockDetails.md, src/zksync/actions/getBlockDetails.ts, site/pages/zksync/actions/getTransactionDetails.md, src/zksync/actions/getL1BatchDetails.ts, src/zksync/actions/getDefaultBridgeAddresses.ts, src/zksync/actions/getLogProof.ts, site/pages/zksync/actions/getL1BatchBlockRange.md, src/zksync/actions/getTransactionDetails.ts, src/zksync/actions/getL1BatchBlockRange.ts, src/zksync/actions/estimateFee.test.ts, src/zksync/actions/getRawBlockTransaction.ts, site/pages/zksync/actions/getLogProof.md, src/zksync/actions/getAllBalances.test.ts, site/pages/zksync/actions/getAllBalances.md, src/zksync/actions/getAllBalances.ts, src/zksync/actions/estimateGasL1ToL2.ts, src/zksync/types/block.ts, src/zksync/actions/getRawBlockTransaction.test.ts, src/zksync/actions/estimateFee.ts, site/sidebar.ts, src/zksync/actions/getLogProof.test.ts, src/zksync/index.ts, src/zksync/types/eip1193.ts, src/zksync/types/transaction.ts, site/pages/zksync/actions/estimateFee.md, site/pages/zksync/actions/estimateGasL1ToL2.md, src/zksync/decorators/publicL2.test.ts, test/src/zksync.ts, src/zksync/decorators/publicL2.ts

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

Copy link

changeset-bot bot commented Apr 22, 2024

🦋 Changeset detected

Latest commit: d40986d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Apr 22, 2024

@nikola-bozin-txfusion is attempting to deploy a commit to the wevm Team on Vercel.

A member of the Team first needs to authorize it.

@nikola-bozin-txfusion
Copy link
Contributor Author

Hey guys, did you have time to check this out?

@jxom jxom force-pushed the nikola-bozin-txfusion-zks-namespace branch from 6a4832e to d40986d Compare May 19, 2024 23:30
@jxom jxom merged commit c2c0791 into wevm:main May 19, 2024
8 of 16 checks passed
@jxom jxom deleted the nikola-bozin-txfusion-zks-namespace branch May 19, 2024 23:31
@github-actions github-actions bot mentioned this pull request May 19, 2024
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

2 participants