Merge pull request #47 from wagesj45/codex/fix-per-domain-charts-loading-bug

Fix chart reload bug
This commit is contained in:
Jordan Wages 2025-07-19 16:42:18 -05:00 committed by GitHub
commit bf09af6b5e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -171,6 +171,15 @@
}); });
} }
function destroyCharts(container) {
container.querySelectorAll('canvas').forEach(c => {
const chart = Chart.getChart(c);
if (chart) {
chart.destroy();
}
});
}
function loadReports() { function loadReports() {
let path; let path;
let container; let container;
@ -183,6 +192,7 @@
} else { } else {
container = containers.domain; container = containers.domain;
if (!currentDomain) { if (!currentDomain) {
destroyCharts(container);
container.innerHTML = '<p>Select a domain</p>'; container.innerHTML = '<p>Select a domain</p>';
return; return;
} }
@ -192,6 +202,7 @@
fetch(path + '/reports.json') fetch(path + '/reports.json')
.then(r => r.json()) .then(r => r.json())
.then(reports => { .then(reports => {
destroyCharts(container);
container.innerHTML = ''; container.innerHTML = '';
reports.forEach(rep => { reports.forEach(rep => {
fetch(path + '/' + rep.html) fetch(path + '/' + rep.html)