-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
New Setting: Disable used percentage #2136
New Setting: Disable used percentage #2136
Conversation
Hi @o1egl , any news for this PR? |
Interested in this PR? |
I'd like to see this merged. Perhaps also create a PR in the documentation repo to document this new functionality? |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Interested in this PR? |
@o1egl , @hacdias , @Equim-chan ? |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Could @o1egl have a look, please? At least one positive or negative return? |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Please @hacdias / @o1egl / @ramiresviana , any feedback? Several issues and PRs being closed, and this one without any feedback |
It seems like devs are probably too busy with other things to look at new features, evidenced by the fact that many recent merges are just tiny bug fixes or translation updates (a response of any kind would be helpful though). While I would have liked to have your feature merged, perhaps it's time to maintain your own fork with your own features and just pull from upstream when needed. |
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.
Could you please resolve the merge conflicts and fix the linting errors?
@o1egl conflicts and linting erros fixed. |
265372c
to
0a2d740
Compare
@o1egl I didn't know the commit policy when I opened the PR. Do I need to rewrite all commits to proceed with the merge? |
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.
When display usage is disabled, the API request is still sent to the server. I think we shouldn't do this
…browser#2184) Removes [ansi-html](https://github.com/Tjatse/ansi-html). It's no longer used after updating ancestor dependency [webpack-dev-server](https://github.com/webpack/webpack-dev-server). These dependencies need to be updated together. Removes `ansi-html` Updates `webpack-dev-server` from 3.11.2 to 3.11.3 - [Release notes](https://github.com/webpack/webpack-dev-server/releases) - [Changelog](https://github.com/webpack/webpack-dev-server/blob/v3.11.3/CHANGELOG.md) - [Commits](webpack/webpack-dev-server@v3.11.2...v3.11.3) --- updated-dependencies: - dependency-name: ansi-html dependency-type: indirect - dependency-name: webpack-dev-server dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
0a2d740
to
537258b
Compare
@o1egl , I maked rebase for fix lint-commits, and fixed request API when config is disabled. |
Thanks @o1egl for this approval! I'm happy to have contributed to this project. |
Description
🚨 Before submitting your PR, please read community, and indicate which issues (in any of the repos) are either fixed or closed by this PR. See GitHub Help: Closing issues using keywords.
Further comments
Hi! I propose this PR to meet a need that we had and could be useful to the community.
Hiding the disk usage percentage graph was necessary to omit the total size of an NFS volume in Kubernetes, which could be misinterpreted or misused by users.
I built the new configuration entirely based on the existing
branding.disableExternal
.I remain at your disposal for further clarifications, and I await its approval and publication.