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: implement SDK with provider injection and connection management #5

Merged
merged 13 commits into from
Sep 6, 2024

Conversation

jinoosss
Copy link
Member

@jinoosss jinoosss commented Sep 3, 2024

What type of PR is this?

  • feature

What this PR does:

This PR introduces a significant update to the SDK, focusing on improved provider management and connection handling.

The core changes include:

SDK Implementation:

  • Developed a robust SDK class to manage wallet interactions, including connection and disconnection operations.
  • Added methods for common wallet functions, such as getAccountInfo and sendTransaction.

Provider Injection:

  • Implemented provider injection into the SDK class, allowing for flexibility in using different wallet providers
    (e.g., AdenaWalletProvider).
  • Providers are now managed through a ConnectionManager class that handles connection state and events.

Connection Management:

  • Added a ConnectionManager class to manage connection state and emit events for connection changes
    (connected and disconnected).
  • Provided methods to register and unregister event listeners for connection state changes.

@jinoosss jinoosss force-pushed the ADN-602-implement-the-sdk-module branch from 0043fbc to 195da81 Compare September 4, 2024 02:57
@jinoosss jinoosss marked this pull request as ready for review September 4, 2024 02:57
@jinoosss jinoosss merged commit b9d9957 into main Sep 6, 2024
3 checks passed
@jinoosss jinoosss deleted the ADN-602-implement-the-sdk-module branch September 6, 2024 06:38
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.

1 participant