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

Fix vote lock display, conditionally disable recovery, improve modal #4840

Open
3 tasks
traumschule opened this issue Apr 21, 2024 · 0 comments
Open
3 tasks
Labels
enhancement New feature or request scope:app/common Features related to the App or common components

Comments

@traumschule
Copy link
Contributor

traumschule commented Apr 21, 2024

Hello! Today i want to draw attention to three more or less cosmetic bugs that can lead to confusion.

When a user chooses to recover a vote lock the transaction fails.

  • The displayed vote is 2 years old.
  • The recover button should be disabled due to a more recent vote lock.
  • The displayed modal copy "Unstaking has been forbidden for the user (at least for now)" sounds more drastic than it has to.

unstakingforbidden2

Below the more recent vote by the same stake for a different candidate in the last election:
recentvote

Solution

To fix the above i suggest the following changes:

Tx details

For reference all displayed information

RECOVERABLE LOCKS:
Voting

LOCK DATE
30/12/2022, 05:42 PM WET

300,453 block


[Modal]
Oh no! Failure
Unstaking has been forbidden for the user (at least for now)

[Tx poopup]
Transaction failed
Error code: referendum.UnstakingForbidden

error.ts:1 Transaction error: Withdraw, TransactionFeePaid, ExtrinsicFailed
on network: wss://rpc.joystream.org:9944
in block: 0xb6f2b0410bca2271c0ed0f5bc08eb288e004d30562c13cfc5161190084500ed3
more details at: https://polkadot.js.org/apps/?rpc=wss://rpc.joystream.org:9944#/explorer/query/0xb6f2b0410bca2271c0ed0f5bc08eb288e004d30562c13cfc5161190084500ed3
@traumschule traumschule added enhancement New feature or request scope:app/common Features related to the App or common components labels Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request scope:app/common Features related to the App or common components
Projects
None yet
Development

No branches or pull requests

1 participant