-
Notifications
You must be signed in to change notification settings - Fork 4
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
Integrate web3 so that users don't have to type in their ETH address when shapeshifting #26
Comments
FYI web3/web3.js#905 |
These seems like a fun one. New to web3 so I'll be getting some education. 🎓 |
Seems like our little exchange form is submitting on successful wallet address validation which is fired in a debounced onchange handler. Since there is no submit button, the web3 enabled user will be stuck experiencing one of two scenarios:
To address this we can:
I'm personally prone to option 2, but am open to any suggestions. |
Hmmmm... Honestly if they don't have to type or copy-paste then I think both of those interactions are kind of awkward. My vote is that we stick to the conventions we're already used to seeing on web3 sites - the input box becomes a dropdown list. If the user has web3, we should also be triggering the transaction for them to send to the deposit address. Basically right now, people have to type in ETH address, then copy the deposit address, then go open their wallet and manually enter the transaction. For web3 users, we have all the APIs available so that they can simply:
Thoughts? And yeah, this should be a really fun one! |
@cooper care to show me an example of this user flow in the wild? Thanks! 😁 |
Yeah, examples off the top of my head are etheroll.com and etherdelta.github.io Etherdelta has a dropdown to choose an address. Etheroll just chooses the first one for you. Also, on Etheroll, when you place a bet, its creating an ETH transaction pre-filled for you with the right data, and then asks you to sign it via web3. we could do the same. |
Looking at these two sites I'm thinking we should always do dropdown. My reasoning being that selecting an address will act as the submit action, triggering address validation and transaction prefilling. Seem good? |
Yeah, exactly what I had in mind. |
No description provided.
The text was updated successfully, but these errors were encountered: