FDS2610/monitor
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request 'Buero-Visualisierung: irrefuehrenden "API nicht erreichbar"-Banner nach erfolgreichem Live-Fetch beheben' (#96) from fds2610-bot/monitor:main into main

Browse source 
Reviewed-on: https://codeberg.org/fds2610/monitor/pulls/96
This commit is contained in:
fds2610 2026-09-23 08:50:14 +02:00
commit 6a35712d66
1 changed files with 32 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

44
html/js/office-render.js
View file

@ -2626,9 +2626,34 @@ async function boot() {
} }
async function refresh() { async function refresh() {
// Issue #94, zweiter Anlauf: der erste Fix (siehe Kommentar unten bei renderOfficeScene())
// sorgte zwar dafuer, dass der Banner ueberhaupt erscheint, hat aber "API nicht erreichbar"
// faelschlich auch dann gezeigt, wenn loadLiveData() laengst erfolgreich war und erst das
// anschliessende Verarbeiten/Zeichnen der Live-Daten scheiterte - Produktion meldete dann
// trotz per curl nachweislich erreichbarer API weiterhin Unerreichbarkeit. Deshalb jetzt
// zwei getrennte try-Bloecke: nur ein Fehlschlag von loadLiveData() selbst ist ein
// Erreichbarkeitsproblem und darf den Demo-Fallback samt Banner ausloesen.
let data;
try { try {
const data = await loadLiveData(); data = await loadLiveData();
} catch (err) {
console.warn("[office] Live-API nicht erreichbar, zeige Demo-Daten.", err);
// setApiBanner() steht in einem eigenen finally: renderOfficeScene() zeichnet bei jedem
// Refresh die komplette Szene neu und kann durch einen Fehler in noch unbekannten
// Datenkombinationen scheitern (siehe Issue #94 - Produktion zeigte trotz nicht
// erreichbarer API keinen Banner). Ohne das finally wuerde ein solcher Fehler die
// wichtigere Meldung "API nicht erreichbar" mit sich reissen, statt sie anzuzeigen.
try {
renderOfficeScene(container);
} finally {
setApiBanner(true);
}
return;
}
setApiBanner(false);
lastLiveData = data; lastLiveData = data;
try {
if (previousTickets) { if (previousTickets) {
const previousById = new Map(previousTickets.map((ticket) => [ticket.id, ticket])); const previousById = new Map(previousTickets.map((ticket) => [ticket.id, ticket]));
@ -2684,19 +2709,12 @@ async function boot() {
resolvePendingAgentTicketMoves(data.agents); resolvePendingAgentTicketMoves(data.agents);
previousTickets = data.tickets; previousTickets = data.tickets;
renderLiveScene(data); renderLiveScene(data);
setApiBanner(false);
} catch (err) { } catch (err) {
console.warn("[office] Live-API nicht erreichbar, zeige Demo-Daten.", err); // Die API war erreichbar (loadLiveData() oben ist bereits durchgelaufen) - ein Fehler
// setApiBanner() steht in einem eigenen finally: renderOfficeScene() zeichnet bei jedem // hier ist ein Render-/Verarbeitungsbug, kein Erreichbarkeitsproblem. Der Banner bleibt
// Refresh die komplette Szene neu und kann durch einen Fehler in noch unbekannten // deshalb aus (siehe setApiBanner(false) oben); der naechste Poll-Zyklus versucht es
// Datenkombinationen scheitern (siehe Issue #94 - Produktion zeigte trotz nicht // erneut, sobald startRealtimeUpdates() ihn ausloest.
// erreichbarer API keinen Banner). Ohne das finally wuerde ein solcher Fehler die console.error("[office] Fehler beim Verarbeiten/Zeichnen der Live-Daten.", err);
// wichtigere Meldung "API nicht erreichbar" mit sich reissen, statt sie anzuzeigen.
try {
renderOfficeScene(container);
} finally {
setApiBanner(true);
}
} }
} }