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

CSUB-1019 Change the staking dashboard "waiting" label on the Validator page to Active + number #72

Merged
merged 6 commits into from
Jan 26, 2024

Conversation

frankli-dev
Copy link
Contributor

No description provided.

@frankli-dev frankli-dev self-assigned this Jan 23, 2024
Copy link
Contributor

Test before merge -> Staking Dashboad Preview!

Copy link
Contributor

@atodorov atodorov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is how it looks with only 1 validator - Alice:
Screenshot 2024-01-24 at 16-08-34 Creditcoin Staking Dashboard Creditcoin Staking (CTC)

This is how the dashboard looks with another validator added (1 era elapsed after adding it), which is still not in the list of active validators:
Screenshot 2024-01-24 at 16-09-02 Creditcoin Staking Dashboard Creditcoin Staking (CTC)

Screenshot 2024-01-24 at 16-09-25 Polkadot_Substrate Portal

Note that by default this page has an "Active validators" filter applied to it, and even when clearing the filter the second validator doesn't always show up:
image

Copy link
Contributor

@atodorov atodorov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review & test round #2

✔️ Validators are shown with the label "Active / X CTC":
01_only_Alice_shows_0_CTC

✔️ A second validator which has indicated their desire to validate, but is still waiting for the current era to elapse (and possible be selected in the validator set) is also shown as soon as they are chosen as part of the active validation set:
02_bob_is_still_waiting

❗ Clicking the "Clear" button doesn't seem to affect anything IMO. I have no idea how it is supposed to work and digging through the code to find that out seems like too much effort in the context of this ticket.

✔️ Having nominators articipate in the mix increases the amount of CTC shown as soon as these nominations become active.

❌ The CTC amount shown does not always sync properly and sometimes it shows "0 CTC". The easiest way to reproduce that is to refresh the page several times subsequently. It is probably unrelated to these actual changes.

@frankli-dev
Copy link
Contributor Author

@atodorov I've fixed the last error the above. (0 CTC sometimes error)

@frankli-dev frankli-dev force-pushed the fix/csub-1019-waiting branch 2 times, most recently from 385eb06 to 979dd31 Compare January 25, 2024 19:55
@frankli-dev
Copy link
Contributor Author

Also fixed the filter.

@frankli-dev frankli-dev force-pushed the fix/csub-1019-waiting branch from 3e37a8b to 05639ae Compare January 25, 2024 20:45
@frankli-dev frankli-dev requested a review from atodorov January 26, 2024 17:09
@atodorov
Copy link
Contributor

❗ Clicking the "Clear" button doesn't seem to affect anything IMO. I have no idea how it is supposed to work and digging through the code to find that out seems like too much effort in the context of this ticket.

🟢 : Clicking the "Clear" button while Bob is still not in the active validator set now works for me.

Screenshot 2024-01-26 at 19-13-47 Creditcoin Staking Dashboard Creditcoin Staking (CTC)

❌ ^^^ during the time in which Bob hasn't made it into the active validator set he is shown as "Active / 0 CTC", see screenshot.

❌ The CTC amount shown does not always sync properly and sometimes it shows "0 CTC". The easiest way to reproduce that is to refresh the page several times subsequently. It is probably unrelated to these actual changes.

🟢 this seems to have been fixed.

Copy link
Contributor

@atodorov atodorov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like it needs a rebase to account for commits from #74.

@frankli-dev frankli-dev force-pushed the fix/csub-1019-waiting branch from 05639ae to 758bac5 Compare January 26, 2024 17:23
@nathanwhit nathanwhit merged commit 3f5c8a0 into dev Jan 26, 2024
11 checks passed
juanma1996 pushed a commit that referenced this pull request Feb 16, 2024
…or page to Active + number (#72)

* fix: Zero era status

* fix: Not in this branch

* fix: Remove waiting on era status

* fix: Show validators even they don't have nominators

* fix: Update component on total stake change

* fix: Clear filter

---------

Co-authored-by: Frank Li <frank.li@gluwa.com>
juanma1996 pushed a commit that referenced this pull request Apr 4, 2024
…or page to Active + number (#72)

* fix: Zero era status

* fix: Not in this branch

* fix: Remove waiting on era status

* fix: Show validators even they don't have nominators

* fix: Update component on total stake change

* fix: Clear filter

---------

Co-authored-by: Frank Li <frank.li@gluwa.com>
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 this pull request may close these issues.

None yet

4 participants