Skip to content

Add Polkadot integration #44

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

0binna-oss
Copy link

@0binna-oss 0binna-oss commented Apr 6, 2025

Description

The Polkadot chain integration adds support for polkadot to the multichain-crypto-wallet. Key features include:

  1. Initialization:

Establishes a connection to the polkadot network using a WebSocket RPC endpoint.

  1. Wallet Generation:

Creates new wallets with sr25519 key pairs.

  1. Balance Retrieval:

fetches account balances with address validation using ss58 format.

  1. Disconnection:

Cleans up API and keyring instances to ensure proper resource management.

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist:

  • I have selected the correct base branch.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • Any dependent changes have been merged and published in downstream modules.
  • I ran npm run test and my test cases cover all the lines and branches of the added code.
  • I ran npm run build with success.
  • I have tested my code on the live network.
  • I have checked the Deploy Preview and it looks correct.

@iamnotstatic iamnotstatic added the enhancement New feature or request label Apr 12, 2025
@iamnotstatic iamnotstatic self-requested a review April 12, 2025 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants