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-1116: Fetching Member List.... is shown on Members tab when pool member is 0 #140

Merged
merged 7 commits into from
Apr 18, 2024

Conversation

frankli-dev
Copy link
Contributor

No description provided.

Copy link
Contributor

Test before merge -> Staking Dashboad Preview!

@frankli-dev frankli-dev self-assigned this Apr 11, 2024
@Dante9988
Copy link

I have reviewed the changes and potential impact by manually creating a pool
When pool has been created on testnet with ID 81/82
After syncing with pool nominations and switching accounts in between Pool members suddenly becomes 1 and when checking pool members "No Members" message has shown
See the video for details

PoolMemberBecomes1AfterCreating.mov

@gaquarius gaquarius merged commit 88736f2 into dev Apr 18, 2024
11 checks passed
@gaquarius gaquarius deleted the csub-1116 branch April 18, 2024 21:10
@atodorov atodorov restored the csub-1116 branch April 19, 2024 08:45
@atodorov atodorov deleted the csub-1116 branch April 19, 2024 08:46
atodorov added a commit that referenced this pull request Apr 19, 2024
This reverts commit bc5ed64 from the
original #140

because it is not necessary!

The failures are with the markdown-link-check linter which is not influenced by
the .lycheeignore config file as seen in the logs for the original commit:
https://github.com/gluwa/creditcoin3-staking-dashboard/actions/runs/8744452909/job/23997350187

^^^ no failures reported by Lychee here
atodorov added a commit that referenced this pull request Apr 19, 2024
This reverts commits
f00f6e0,
8a446ac,
b146e14
from the original
#140

A better way to fix this is by providing a config file and ignoring the
URL in question as shown at
https://github.com/tcort/markdown-link-check?tab=readme-ov-file#config-file-format
atodorov added a commit that referenced this pull request Apr 29, 2024
This reverts commit bc5ed64 from the
original #140

because it is not necessary!

The failures are with the markdown-link-check linter which is not influenced by
the .lycheeignore config file as seen in the logs for the original commit:
https://github.com/gluwa/creditcoin3-staking-dashboard/actions/runs/8744452909/job/23997350187

^^^ no failures reported by Lychee here
atodorov added a commit that referenced this pull request Apr 29, 2024
This reverts commits
f00f6e0,
8a446ac,
b146e14
from the original
#140

A better way to fix this is by providing a config file and ignoring the
URL in question as shown at
https://github.com/tcort/markdown-link-check?tab=readme-ov-file#config-file-format
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.

4 participants