Skip to content

web: Make number of active nodes more clear #116

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

Merged
merged 1 commit into from
Apr 17, 2025

Conversation

mike-petrov
Copy link
Contributor

closes #115

Signed-off-by: Mikhail Petrov mike@nspcc.ru

@mike-petrov mike-petrov self-assigned this Apr 16, 2025
@mike-petrov mike-petrov requested a review from carpawell April 16, 2025 15:30
Signed-off-by: Mikhail Petrov <mike@nspcc.ru>
@mike-petrov mike-petrov force-pushed the feature/115-make_statusmsg_more_clear branch from 7fc2e2a to 65770e2 Compare April 17, 2025 13:53
@@ -162,10 +162,10 @@ async def main():

if node_mainnet_count <= 3:
output['status']['mainnet'] = "Severe"
output['statusmsgs']['mainnet'].append(f"{node_mainnet_count} / 5 nodes is available")
output['statusmsgs']['mainnet'].append(f"{node_mainnet_count} out of 5 nodes {'is' if node_mainnet_count == 1 else 'are'} available")
Copy link
Member

Choose a reason for hiding this comment

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

i do not want believe that we ever see "0" but if yes, i think it is also "is"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@roman-khimov roman-khimov merged commit 11d5b23 into master Apr 17, 2025
2 checks passed
@roman-khimov roman-khimov deleted the feature/115-make_statusmsg_more_clear branch April 17, 2025 14:08
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.

Make number of active nodes more clear
3 participants