-
-
Notifications
You must be signed in to change notification settings - Fork 556
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
Add "Clients (over time)" graph #583
Conversation
scripts/pi-hole/js/index.js
Outdated
@@ -642,7 +728,7 @@ $(document).ready(function() { | |||
var m = parseInt(time[2], 10) || 0; | |||
var from = padNumber(h)+":"+padNumber(m-5)+":00"; | |||
var to = padNumber(h)+":"+padNumber(m+4)+":59"; | |||
return "Forward destinations from "+from+" to "+to; | |||
return "Client activity from "+from+" to "+to; |
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.
You flipped the forward destinations and client activity hover text
scripts/pi-hole/js/index.js
Outdated
@@ -680,6 +766,64 @@ $(document).ready(function() { | |||
}); | |||
|
|||
// Pull in data via AJAX | |||
updateClientsOverTime(); |
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.
This should be switched with the forward destinations call on line 827.
scripts/pi-hole/js/index.js
Outdated
var m = parseInt(time[2], 10) || 0; | ||
var from = padNumber(h)+":"+padNumber(m-5)+":00"; | ||
var to = padNumber(h)+":"+padNumber(m+4)+":59"; | ||
return "Forward destinations from "+from+" to "+to; |
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.
Client activity
Signed-off-by: DL6ER <dl6er@dl6er.de>
Signed-off-by: DL6ER <dl6er@dl6er.de>
With the change in excluding clients from the client graph in the corresponding FTL PR, this PR should change the settings page info for the excluded domains section. |
…like we do it already for the main graph) Signed-off-by: DL6ER <dl6er@dl6er.de>
…o new/clientsovertime
By submitting this pull request, I confirm the following (please check boxes, eg [X] - no spaces) Failure to fill the template will close your PR:
Please submit all pull requests against the
development
branch. Failure to do so will delay or deny your requestHow familiar are you with the codebase?:
10
This adds an over time graph for the individual clients that used the Pi-hole as their DNS server.
The "design" of the new graph has still to be improved - this is only a proof-of-concept screenshot:
Important: Needs up-to-date version of
FTL
on custom branchnew/clientsovertime
This template was created based on the work of
udemy-dl
.