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

Update dapp-psm smart wallet provisioning UX to accommodate auto-wallet provisioning change #116

Closed
otoole-brendan opened this issue Nov 30, 2023 · 0 comments · Fixed by #119
Assignees

Comments

@otoole-brendan
Copy link

otoole-brendan commented Nov 30, 2023

What is the Problem Being Solved?
In preparation for auto wallet provisioning where absent (Agoric/agoric-sdk#8558) - there are dapp-psm changes we can make to accommodate this new flow/UX.

In current flow, a user is prompted with the 'You need to provision a wallet dialog that will cost 10BLD' if they connect Keplr on the app and no wallet is detected.

Description of the Design
Once #8558 is deployed - the desired flow will change to:

User visits psm.inter.trade
User connects Keplr
User configures swap trade
User hits 'swap'
User is prompted with dialog that they will need 1IST in order to provision the wallet before being able to create the vault. They can opt to 'Proceed' (which will attempt to send the txn and should succeed if they have the IST required to provision and fail if they do not (Question @samsiegart can we customize the 'you don't have enough IST to provision a wallet' error message?)) , 'Get IST' or 'Cancel'. 'Get IST' brings up the Leap Elements modal to enable them to easily onboard IST. Hitting Cancel returns them to vault setup view.
Dialog Text + Design:

Title: 'Smart Wallet Required'
'To interact with contracts on the Agoric chain, a smart wallet must be created for your account. As an anti-spam measure, you will need 1 IST to fund its provision which will be deposited into the community fund. Click 'Proceed' to provision wallet and submit transaction.'

Security Considerations
N/A

Test Plan
Will be smoke tested

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 a pull request may close this issue.

2 participants