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
Tweaks to functions in utils.sh and refactored usages in webpage.sh #4653
Conversation
…(adjust test to suit) Add a removeKey function with test update webpage.sh to reference functions in utils.sh (this can likely be abstracted/refactored further) Signed-off-by: Adam Warner <me@adamwarner.co.uk>
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.
Can you make sure that you don't use any bashisms? I'm working on POSIX compliance so we can run in non-bash native distributions. (i.e. Alpine).
b4efb36
to
bedf472
Compare
Signed-off-by: Adam Warner <me@adamwarner.co.uk>
bedf472
to
59fc380
Compare
678ce56
to
ee0f0bd
Compare
ee0f0bd
to
b8f703f
Compare
b8f703f
to
7fa8cdd
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.
L263 and 273 in pihole
can be expressed with the new functions form utils.sh
as well
Signed-off-by: Adam Warner <me@adamwarner.co.uk>
Signed-off-by: Adam Warner <me@adamwarner.co.uk>
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/pi-hole-ftl-v5-15-web-v5-12-and-core-v5-10-released/54987/1 |
By submitting this pull request, I confirm the following:
git rebase
)What does this PR aim to accomplish?:
Move some functionality out of webpage.sh and into utils.sh - keep the wrapper functions in webpage.sh for now as docker is currently using them - but these can go once we update docker container