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
Use automatic number formatting #2973
Conversation
c3c660f
to
25255d5
Compare
Hmm, how to define |
Short answer: you don't :) Longer answer: anything in the vendor directories are third party libraries, which we shouldn't be making changes to. Which part has issues that would need changes in charts? |
To resolve these CI warnings, check out web/scripts/pi-hole/js/charts.js Line 8 in f79a404
and make this something like
|
Ah, apologies - ignore my comment. I made it without looking at the files changed, I didn't realise we also had a script named |
Signed-off-by: Emir SARI <emir_sari@icloud.com>
I think this will probably cover many things, and will make it easier to spot missed ones. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
@bitigchi please fix merge conflicts |
Hi, I won't be able to return to this for a while, I am sorry (burnt out totally). In the meantime, if someone wants to fix this, I'd really appreciate... |
Existing merge conflicts have not been addressed. This PR is considered abandoned. |
Thank you for your contribution to the Pi-hole Community!
Please read the comments below to help us consider your Pull Request.
We are all volunteers and completing the process outlined will help us review your commits quicker.
Please make sure you
What does this PR aim to accomplish?:
Uses automatic number formatting that uses system locale settings.
How does this PR accomplish the above?:
By using the appropriate internalisation API's.
By submitting this pull request, I confirm the following:
git rebase
)