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

Ton Connect implementation #7

Closed
wants to merge 2 commits into from

Conversation

stasionok
Copy link

@stasionok stasionok commented Feb 11, 2025

Relates to

#2984

What does this PR do?

This plugin add support of Ton Connect protocol

What kind of change is this?

  • Plugin allow make connection with all supported ton wallets via Ton Connect protocol
  • Make ability to use connectet wallet to make any transaction etc via provider
  • Allow multiwallet connection and disconnection
  • Can show list connected addresses/wallet

Detailed testing steps

Ask your character as example

  • Let connect ton with Tonkeeper
    It show you connection link for tonkeeper and show QRCode to connect from other device

After connection you can ask

  • Show me connected wallets
    And you get list connected

Now you can disconnect any wallet you need

  • Disconnect my wallet UV4fdsof34h....

More over you can use this connection from any place. Just call

import { TonConnectWalletProvider } from "@elizaos/plugin-ton";

const walletProvider  = new WalletProvider(runtime);
const wallet = walletProvider.getWalletClient();

or

import { TonConnectWalletProvider } from "@elizaos/plugin-ton";

const walletProvider  = new WalletProvider(runtime);
const wallet = walletProvider.getWalletClient(address);

@fivestarsfi fivestarsfi self-requested a review February 21, 2025 20:52
Copy link
Collaborator

@fivestarsfi fivestarsfi left a comment

Choose a reason for hiding this comment

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

Great code, but there are a few comments. Firstly, there is very little documentation without proof, and during local testing I was unable to connect the wallet using a QR code.

@fivestarsfi
Copy link
Collaborator

Also, this Issue has been made and has already been merge. There is also no sendTransaction function. Unfortunately, we can't accept it because it doesn't work and is already merged it.

@fivestarsfi fivestarsfi added bug Something isn't working duplicate This issue or pull request already exists labels Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants