Skip to content

[ID-3696] refactor: move linking wallet example to zkevm api directory #465

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

Merged
merged 1 commit into from
May 13, 2025

Conversation

nattb8
Copy link
Collaborator

@nattb8 nattb8 commented May 13, 2025

Summary

This PR moves the Link Wallet example from the Passport package to the ZkEvmApi directory. While the example originally lived under Passport, it primarily depends on the ZkEvmApi package for its functionality. The only requirement from Passport is an access token, which can be manually provided by the user if needed.

This change ensures that the sample code in the Passport package (Samples~ folder) remains self-contained and compiles successfully when imported, without requiring users to install the ZkEvmApi package.

Customer Impact

This change improves compatibility by ensuring the Passport sample code (Samples~ folder) works out of the box without requiring the ZkEvmApi package. No breaking changes - the moved example is still available under ZkEvmApi samples (this and this).

@nattb8 nattb8 requested a review from a team May 13, 2025 06:36
@nattb8 nattb8 force-pushed the refactor/link-wallet-zkevmapi branch from a9d802e to db7f60f Compare May 13, 2025 06:40
@nattb8 nattb8 merged commit 16c07fc into main May 13, 2025
23 of 24 checks passed
@nattb8 nattb8 deleted the refactor/link-wallet-zkevmapi branch May 13, 2025 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants