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 Nest Wallet #77

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

Conversation

matthewliu10
Copy link

Add Nest Wallet

Supports

Supported features

Tests

a demo dapp with integrated wallet

Integrator contacts

@matthewliu10 matthewliu10 changed the title add nest wallet feat: Add Nest Wallet Aug 2, 2024
@LetikGit
Copy link
Member

LetikGit commented Aug 5, 2024

@matthewliu10 Hello! You have specified android platform, but you have not provided a sse bridge and universal_url.

If you support android platform you should fix your configuration and add the missing fields.

You can find more information here:

About the fields - https://github.com/ton-blockchain/wallets-list?tab=readme-ov-file#description
TON Connect specification - https://github.com/ton-blockchain/ton-connect

@matthewliu10
Copy link
Author

Hello! I previously added android because we support the mobile browser on android. I've removed it now since I guess that doesn't count as support android.

@matthewliu10
Copy link
Author

Hey I made the change to remove android. Could you review again please? @LetikGit

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.

2 participants