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

Validator dashboard QA #4743

Open
thesan opened this issue Jan 22, 2024 · 11 comments
Open

Validator dashboard QA #4743

thesan opened this issue Jan 22, 2024 · 11 comments

Comments

@thesan
Copy link
Member

thesan commented Jan 22, 2024

⚠️ Dependencies

The QA should start only once #4742 and Joystream/joystream#4933 get merged and dev is merged into validator branch one last time.

Context

Designs

Testing environment 👉 https://dao-git-validators-joystream.vercel.app

Scope

Dashboard itself

This should be tested at least partially on Mainnet. There has been some back and forth on how to calculate some of the values and it's important to make sure that non what is displayed is misleading to nominators. Also this page should work well in mobile:

  1. Dashboard statistic headers statistic accuracy:
    image
  2. Dashboard list item accuracy + interaction with the filters/ordering/pagination, etc..
    image
  3. Validator cards layout + data accuracy:
    image
  4. The "Nominate" links:
    image

Binding validator accounts to memberships

It might be easier to test this on a playground. In that case Atlas dev should be fine: https://dao-git-validators-joystream.vercel.app/#/settings?network-config=https://atlas-dev.joystream.org/network/config.json

Regardless you'll need to create validator accounts on the network used for the tests.

  1. Binding a/multiple validators to a new membership.
  2. Binding a/multiple validators to an existing membership.
  3. Verify and unverifiy validator accounts via the CLI (add CLI command verifyValidator joystream#4933) and check their statuses on the dashboard.
@ivanturlakov
Copy link

ivanturlakov commented Jan 25, 2024

Dashboard

Tested on https://dao-git-validators-joystream.vercel.app/ Mainnet

✅ Dashboard statistic headers statistic accuracy

⚠️ Dashboard list item accuracy + interaction with the filters/ordering/pagination, etc..

  • The average APR calculation looks incorrect. If you sort by descending - the first 14 validators' ARPs will be 5%-2%, the rest 1%-0%
  • Let us make social media icons as hyperlinks to related resources
  • Search validation msg (desktop & mobile)
Screenshot 2024-01-24 at 13 05 30 Screenshot 2024-01-24 at 15 32 06

⚠️ Validator cards layout + data accuracy

  • Let us make social media icons as hyperlinks to related resources
  • Customize the width of the mobile version if possible
Screenshot 2024-01-24 at 15 34 14
  • Nominate modal - typo “NoRminate”
Screenshot 2024-01-25 at 13 36 57

@kdembler
Copy link
Member

No tooltip texts:
CleanShot 2024-01-27 at 16 46 31@2x
CleanShot 2024-01-27 at 16 46 25@2x

Unclear "Last" in rewards section. I guess it's last era?
CleanShot 2024-01-27 at 16 46 36@2x

If "All" is selected in filters, there's no text in the select:
CleanShot 2024-01-27 at 16 46 43@2x

Validation message shouldn't appear as I'm typing, only once I submit with Enter:

CleanShot.2024-01-27.at.16.49.40.mp4

When I search for something, and then remove it and confirm with Enter, the search isn't cancelled (I press Enter at the end):

CleanShot.2024-01-27.at.16.50.29.mp4

@eshark9312
Copy link
Contributor

eshark9312 commented Jan 30, 2024

@kdembler @ivanturlakov
I fixed most of the issues, please check with this url.
https://dao-9pwmfuy5v-joystream.vercel.app/

@eshark9312
Copy link
Contributor

eshark9312 commented Jan 30, 2024

@ivanturlakov
I have not fixed the apr calculation yet. (It looks correct to me.)
Could you check again and let me know what the APRs should look?
Total staking is about 120M (almost of the staking comes from nominators), the last month reward is about 1M then the annual reward is 12M.
When the commission is 10%, then the reward for the nominators is 1.2M so the average apr is about 1%.

@ivanturlakov
Copy link

ivanturlakov commented Jan 30, 2024

Retested on https://dao-9pwmfuy5v-joystream.vercel.app/ mainnet
https://dao-git-fork-eshark9312-validator-dashboard-qa-joystream.vercel.app/#/validators mainnet

✅ tooltips
✅ filters/search
✅ Validator card

@eshark9312 thanks!

Pls add a minor fix for Social links
Change Telegram link to "https://t.me/userName"

@kdembler what do you think about the formula for calculating expected Nominator APR? I am a Nominator and have checked my actual average APR and it ranges from 5 to 7% while the expected APR shows 1%

@thesan
Copy link
Member Author

thesan commented Jan 30, 2024

@ivanturlakov thank you for the quick feedback. But the correct environment is: https://dao-git-fork-eshark9312-validator-dashboard-qa-joystream.vercel.app/ (not https://dao-9pwmfuy5v-joystream.vercel.app).

On the APR thanks a lot for checking, it's very important to make sure we are not underestimating those. I'll look some more into it this week. @kdembler @eshark9312 please have look too.

@eshark9312
Copy link
Contributor

eshark9312 commented Jan 30, 2024

@ivanturlakov
You are right. The APR for nominators should be calculated by the following formula:
Yearly Reward * (1 - Commission) / Stake
cc @kdembler @thesan

@ivanturlakov
Copy link

Tested on https://37.27.43.23.nip.io/pioneer/#/settings?network-config=https://37.27.43.23.nip.io/network/config.json

Binding a/multiple validators to a new membership

Trying to add two validators when creating a new user

⚠️ Strange behavior during validation - when adding the first validator, the Create button is inactive.

Screenshot 2024-02-06 at 19 50 00

But if you add an empty field for the second validator, the button is activated.

Screenshot 2024-02-06 at 19 50 09

⚠️ I added the addresses of 2 validators but only one was bound (I suspect the first one on the list). The transaction was also performed for only one validator.

Binding a/multiple validators to an existing membership

⚠️ Strange behavior during validation - when adding the first validator, the Save button is inactive.

Screenshot 2024-02-06 at 20 13 01

But if you add an empty field for the second validator, the button is activated.

Screenshot 2024-02-06 at 20 13 11

⚠️ I added the addresses of 2 validators but only one was bound (I suspect the first one on the list). The transaction was also performed for only one validator.
Screenshot 2024-02-06 at 19 50 23

⚠️ I tried to bind the second validator separately (the first one is already displayed in the profile), but when clicking Save the transaction is not initiated, but immediately shows the success window

@eshark9312
Copy link
Contributor

eshark9312 commented Feb 8, 2024

@ivanturlakov
After you select the validator account(or paste the address), you have to click add button, if not it is not included to the validator accounts to be bound to your membership.
And the last issue:

I tried to bind the second validator separately (the first one is already displayed in the profile), but when clicking Save the transaction is not initiated, but immediately shows the success window

I fixed it, in this case the save button should be disabled until click the add button.
#4767

@ivanturlakov
Copy link

After you select the validator account(or paste the address), you have to click add button, if not it is not included to the validator accounts to be bound to your membership.

Screenshot 2024-02-08 at 10 10 49

This is not obvious logic. Validator address fields should work like a normal form field - if there is an address there, then it is valid. The add button should only be responsible for adding a new validator address field. If we leave everything as it is, we need to add an explanation (after entering the validator, press the plus button). @kdembler @thesan what you think?

@ivanturlakov
Copy link

Verify and unverifiy validator accounts via the CLI

Tested on https://37.27.43.23.nip.io/pioneer/#/settings?network-config=https://37.27.43.23.nip.io/network/config.json

Works as expected
✅ Verify tester-6 yarn joystream-cli working-groups:verifyValidator --group=membership 1
✅ unVerify tester-6 yarn joystream-cli working-groups:verifyValidator --group=membership 1
✅ Verify tester-5 yarn joystream-cli working-groups:verifyValidator --group=membership 0

Screenshot 2024-02-14 at 20 36 26 Screenshot 2024-02-14 at 20 50 05

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants