Skip to content

Add Guarda Wallet #70

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add Guarda Wallet #70

wants to merge 1 commit into from

Conversation

eq1337
Copy link

@eq1337 eq1337 commented Jul 16, 2025

No description provided.

@zorro11639 zorro11639 added the wallet support Request support for new wallet label Jul 28, 2025
octavio12345300
octavio12345300 approved these changes Aug 4, 2025
View reviewed changes
Copy link

@octavio12345300 octavio12345300 left a comment

Choose a reason for hiding this comment

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

list

@zorro11639
Copy link
Contributor

zorro11639 commented Aug 5, 2025 •
edited
Loading

Thanks for your PR. We will test the wallet function and then publish the package.

Before we start to test, could you help me to confirm the Wallet features ? @octavio12345300 @eq1337

What platform does Guarda wallet support ?

  • Android app
  • iOS app
  • Extension

What functions does Guarda wallet support ?

  • signMessage
  • signTransaction
  • multiSign
  • Deeplink to open Ctrl Wallet App

What events does Guarda wallet support ?

  • accountsChanged

@zorro11639
Copy link
Contributor

Hi, there are some issues with Guarda extension wallet.

  1. TRX has a precision of 6 decimal places. When I try to transfer 0.000001TRX, the wallet display "1TRX".
  2. The sender and receivers address should be shown in base58 format which starts with "T". @eq1337
Transaction Details

@eq1337
Copy link
Author

eq1337 commented Aug 11, 2025

Hello

What platform does Guarda wallet support ?

  • Extension

What functions does Guarda wallet support ?

  • signMessage
  • signTransaction

octavio12345300
octavio12345300 approved these changes Aug 11, 2025
View reviewed changes
Copy link

@octavio12345300 octavio12345300 left a comment

Choose a reason for hiding this comment

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

List

@eq1337
Copy link
Author

eq1337 commented Aug 11, 2025

Hi, there are some issues with Guarda extension wallet.

Thanks for reporting! I'll investigate the issue with Guarda extension and get back to you with updates

octavio12345300
octavio12345300 approved these changes Aug 11, 2025
View reviewed changes
Copy link

@octavio12345300 octavio12345300 left a comment

Choose a reason for hiding this comment

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

Liat

@eq1337
Copy link
Author

eq1337 commented Aug 12, 2025

Errors and comments have been corrected. Can you check again?

@zorro11639
Copy link
Contributor

Thanks a lot. It works fine.

@zorro11639 zorro11639 moved this from Todo to In Progress in TronWallet Adapter Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@octavio12345300 octavio12345300 octavio12345300 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
wallet support Request support for new wallet
Projects
TronWallet Adapter
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eq1337 @zorro11639 @octavio12345300