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: add TM2 wallet provider type to the SDK #7

Merged
merged 8 commits into from
Sep 9, 2024

Conversation

jinoosss
Copy link
Member

@jinoosss jinoosss commented Sep 5, 2024

What type of PR is this?

  • feature

What this PR does:

This PR introduces a new tm2 wallet provider type to the SDK, enabling support for the tm2 wallet provider. This addition extends the SDK’s capabilities by allowing it to interact with different types of wallet providers.

Key Changes:

  1. Added tm2 Wallet Provider:
  • The tm2 wallet provider has been added to the walletProviders.
  • Logic for interacting with the tm2 provider, such as handling transactions and retrieving account information, has been implemented.
  1. Maintaining Consistency with Existing Providers:
  • The tm2 provider follows the existing structure of wallet providers in the SDK.
  • Implemented the tm2 wallet provider to enable social login based on web3-auth
  • Ensured that the implementation is clean and scalable, keeping in line with the current design patterns.

@jinoosss jinoosss force-pushed the ADN-593-sdk-social-account-management-module branch from 92395ff to 03282fc Compare September 6, 2024 06:40
@jinoosss jinoosss marked this pull request as ready for review September 6, 2024 07:10
Copy link

@onlyhyde onlyhyde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jinoosss jinoosss merged commit ee6954b into main Sep 9, 2024
3 checks passed
@jinoosss jinoosss deleted the ADN-593-sdk-social-account-management-module branch September 9, 2024 00:47
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.

3 participants