Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

Basic UI that allows staking but with minimal/poor user experience #196

Merged
merged 7 commits into from
Jul 11, 2022

Conversation

shelbyd
Copy link
Contributor

@shelbyd shelbyd commented Jun 22, 2022

@nikoladimitroff Let me know how you'd like to verify this. I can make a build that points at the devnet and do the necessary chain setup there. Or I could record a video of me using this against a local environment.

This is good enough to deploy with, but not a great UX. It notably doesn't show estimated distribution (as that's non-trivial to calculate and is likely to change from when the user sees it). We can deploy with or without that feature.

#192

@nikoladimitroff
Copy link
Contributor

@shelbyd Let's do a video

@shelbyd
Copy link
Contributor Author

shelbyd commented Jun 22, 2022

Video of most of the relevant flow (actual unstaking didn't capture because linux screen recording...): https://drive.google.com/file/d/1GWTm6rkWJdX5Q0CM9JuM4Jvz_wbaOPWr/view?usp=sharing

Image showing amount unstaked: https://drive.google.com/file/d/1FQ4VJS4sQw95FpTziDbEWj3zM_OKS1g4/view?usp=sharing

@nikoladimitroff
Copy link
Contributor

@shelbyd

  • counting time in blocks is bad for UX, let's put in weeks instead; this might not be technically accurate but it is much better for the user
  • the radiobutton in the UI that says "25 for 100 shares", etc. needs to become "2 weeks for 100 distribution shares (?)" - the (?) being a hover button with an explanation of the shares; I can share a text for that a bit later

@shelbyd
Copy link
Contributor Author

shelbyd commented Jun 27, 2022

@nikoladimitroff Want me to do this in the same PR? It will make the review a bit more complex for you.

Note to me:

  • Hide staking button if staking not available.

@shelbyd
Copy link
Contributor Author

shelbyd commented Jun 28, 2022

@shelbyd shelbyd merged commit c9b9e01 into dev Jul 11, 2022
@shelbyd shelbyd deleted the shelbyd/basic-staking-ui branch July 11, 2022 14:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants