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
Restart speedtest each time needed #169
Conversation
0e1b024
to
8e9c2f4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my opinion, both "Server sponsor" and "Server ID" doesn't really offer a real value to the normal user.
What is the user supposed to do with them? The user should now the command line cli syntax to have relevant information from it.
If we want to expose extra info, it should be hidden by default.
We are in a popover, we cannot display much informations and it is a pity because with the last version of speedtest-cli we could display the ISP, the external public IP, much informations. However I came to the conclusion that it could be a good new network tool, let's remove it for now |
speedtest-cli updated and available in nethserver-testing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good to me, but let's wait Edoardo approval about the JS part
this PR intends to fix
/s
to the value of download/uploadNethServer/dev#6385