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: Integrate ZKSync Wallet for ChainTechnicalStack-Specific Transaction Signing #4914

Merged
merged 77 commits into from
Dec 16, 2024

Conversation

mshojaei-txfusion
Copy link
Contributor

Description

This PR integrates the zksync-ethers wallet implementation for the ZKSync signer strategy. Key changes include:

  • Added zksync-ethers dependency (v5.10.0)
  • Updated ZKSyncSignerStrategy to use ZKSyncWallet instead of standard ethers Wallet
  • Removed outdated TODO comment about ZKSync signer implementation

Drive-by changes

None

Related issues

N/A

Backward compatibility

Yes

Testing

Manual testing

mortezashojaei and others added 30 commits October 16, 2024 15:07
…ocolSigner implementation for flexible cross-chain signer configuration and management
@mergify mergify bot mentioned this pull request Dec 12, 2024
@xeno097 xeno097 added this pull request to the merge queue Dec 16, 2024
github-merge-queue bot pushed a commit that referenced this pull request Dec 16, 2024
…ction Signing (#4914)

### Description

This PR integrates the zksync-ethers wallet implementation for the
ZKSync signer strategy. Key changes include:
- Added zksync-ethers dependency (v5.10.0)
- Updated ZKSyncSignerStrategy to use ZKSyncWallet instead of standard
ethers Wallet
- Removed outdated TODO comment about ZKSync signer implementation

### Drive-by changes

None

### Related issues

N/A

### Backward compatibility

Yes

### Testing

Manual testing

---------

Co-authored-by: Morteza Shojaei <[email protected]>
Co-authored-by: ljankovic-txfusion <[email protected]>
Co-authored-by: ljankovic-txfusion <[email protected]>
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 16, 2024
@xeno097 xeno097 added this pull request to the merge queue Dec 16, 2024
Merged via the queue into hyperlane-xyz:main with commit 98ee79c Dec 16, 2024
29 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants