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: use dialog for add token button #73

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

Conversation

farolanf
Copy link
Contributor

Fix #42

@temptemp3
Copy link
Owner

temptemp3 commented Dec 21, 2023

Screenshot 2023-12-21 at 16 21 10

These move the message down so that there is not a jump in the ui when it appears and disappears.

Maybe what we can do is not have a error message every time an asset is not found.

  • Update asset not found error message to be a less jump ux

@temptemp3
Copy link
Owner

temptemp3 commented Dec 21, 2023

Screenshot 2023-12-21 at 16 37 13
  • Update add confirmation for vsa
    I am thinking we would need to show some asset info and the confirmation should say "Opt in to {ASSETNAME} (VSA:{ASSETID})

@temptemp3
Copy link
Owner

temptemp3 commented Dec 21, 2023

Screenshot 2023-12-21 at 16 43 03

Unable to add asset. After clicking add button nothing happens but there is a failed request.

Asset id: 1290865

@temptemp3
Copy link
Owner

temptemp3 commented Dec 21, 2023

  • Adding arc200 might require some fixing as well
    ARC200:6795477 TacoCoin

@temptemp3
Copy link
Owner

Request changes. Thx!

@farolanf
Copy link
Contributor Author

Maybe what we can do is not have a error message every time an asset is not found

How about this? (No UI jumps)
Screenshot 2023-12-22 174013

Or should we remove the error message?

@temptemp3
Copy link
Owner

temptemp3 commented Dec 22, 2023 via email

@farolanf
Copy link
Contributor Author

When adding the asset by invoking acc.tokenAccept(tokenId) the WalletConnect QRCode modal pops up which is different with the wallet I use to sign in(WalletConnectModalSign).

With the WalletConnect QRCode I'm unable to sign in
Screenshot 2023-12-25 083508

It works with WalletConnectModalSign
Screenshot 2023-12-25 083612

I think it should use whatever wallet provider the user used to sign in, but I haven't found a way to do it. Any directions?

@temptemp3
Copy link
Owner

When adding the asset by invoking acc.tokenAccept(tokenId) the WalletConnect QRCode modal pops up which is different with the wallet I use to sign in(WalletConnectModalSign).

With the WalletConnect QRCode I'm unable to sign in Screenshot 2023-12-25 083508

It works with WalletConnectModalSign Screenshot 2023-12-25 083612

I think it should use whatever wallet provider the user used to sign in, but I haven't found a way to do it. Any directions?

I would look at components/SwapForm it has support for a custom wallet (kibisis) and defly (A-wallet)

@temptemp3
Copy link
Owner

I would also pull in changes from main. It has a web3 modal that looks like it is for A-wallet.

@farolanf
Copy link
Contributor Author

Demo

demo.mp4

@temptemp3
Copy link
Owner

temptemp3 commented Jan 10, 2024 via email

@temptemp3
Copy link
Owner

Demo

demo.mp4

Could you make a demo for voitest adding an arc200?

@farolanf
Copy link
Contributor Author

farolanf commented Jan 16, 2024

Tried adding asset with Kibisis on Voi Testnet but got error:

OperationCanceledError: no response from wallet for "message.enable_request"

Screenshot 2024-01-16 095129

@temptemp3
Copy link
Owner

temptemp3 commented Jan 16, 2024 via email

@farolanf
Copy link
Contributor Author

Kibisis 1.9.0

Also tried after merging with feature/swap, same results.

@temptemp3
Copy link
Owner

temptemp3 commented Jan 16, 2024 via email

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.

Bounty: UI Improvement: Use dialog instead of window.prompt for Add Token button
2 participants