chore: clean up debug logging

This commit is contained in:
SethBurkart123
2026-03-18 09:27:08 +11:00
parent 67f98b13ad
commit 915ce6f5f1
+27 -63
View File
@@ -125,43 +125,33 @@ export async function loadHomePage() {
upcomingItems.classList.remove("loading"); upcomingItems.classList.remove("loading");
} }
const noticesFilterPrefs = prefs.payload.filter( const labelArray = prefs.payload
(item: any) => item.name === "notices.filters", .filter((item: any) => item.name === "notices.filters")
); .map((item: any) => item.value);
const labelArray = noticesFilterPrefs.map((item: any) => item.value);
console.info("[BetterSEQTA+] Notices: prefs", {
hasNoticesFiltersPref: noticesFilterPrefs.length > 0,
noticesFilterPrefs,
labelArrayLength: labelArray.length,
});
const noticeContainer = document.getElementById("notice-container"); const noticeContainer = document.getElementById("notice-container");
if (!noticeContainer) { if (noticeContainer) {
console.warn("[BetterSEQTA+] Notices: notice-container element not found"); if (labelArray.length > 0) {
} else if (labelArray.length > 0) { const dateControl = document.querySelector(
const dateControl = document.querySelector( 'input[type="date"]',
'input[type="date"]', ) as HTMLInputElement;
) as HTMLInputElement; if (dateControl) {
if (dateControl) { dateControl.value = TodayFormatted;
dateControl.value = TodayFormatted; setupNotices(labelArray[0].split(" "), TodayFormatted);
setupNotices(labelArray[0].split(" "), TodayFormatted); }
noticeContainer.classList.remove("loading");
} else { } else {
console.warn("[BetterSEQTA+] Notices: date input not found, cannot setup notices"); noticeContainer.classList.remove("loading");
noticeContainer.innerHTML = "";
const emptyState = document.createElement("div");
emptyState.classList.add("day-empty");
const img = document.createElement("img");
img.src = browser.runtime.getURL(LogoLight);
const text = document.createElement("p");
text.innerText = "No notices available.";
emptyState.append(img, text);
noticeContainer.append(emptyState);
} }
noticeContainer.classList.remove("loading");
} else {
console.info("[BetterSEQTA+] Notices: no notices.filters pref — showing empty state");
noticeContainer.classList.remove("loading");
noticeContainer.innerHTML = "";
const emptyState = document.createElement("div");
emptyState.classList.add("day-empty");
const img = document.createElement("img");
img.src = browser.runtime.getURL(LogoLight);
const text = document.createElement("p");
text.innerText = "No notices available.";
emptyState.append(img, text);
noticeContainer.append(emptyState);
} }
return cleanup; return cleanup;
@@ -247,18 +237,11 @@ async function GetActiveClasses() {
} }
function setupNotices(labelArray: string[], date: string) { function setupNotices(labelArray: string[], date: string) {
console.info("[BetterSEQTA+] Notices: setupNotices called", {
labelArray,
date,
labelCount: labelArray.length,
});
const dateControl = document.querySelector( const dateControl = document.querySelector(
'input[type="date"]', 'input[type="date"]',
) as HTMLInputElement; ) as HTMLInputElement;
const fetchNotices = async (date: string) => { const fetchNotices = async (date: string) => {
console.info("[BetterSEQTA+] Notices: fetchNotices start", { date });
try { try {
const data = settingsState.mockNotices const data = settingsState.mockNotices
? getMockNotices() ? getMockNotices()
@@ -270,15 +253,10 @@ function setupNotices(labelArray: string[], date: string) {
}) })
).json(); ).json();
console.info("[BetterSEQTA+] Notices: fetchNotices response", {
date,
hasPayload: !!data?.payload,
payloadLength: Array.isArray(data?.payload) ? data.payload.length : "n/a",
rawKeys: data ? Object.keys(data) : [],
});
processNotices(data, labelArray); processNotices(data, labelArray);
} catch (error) { } catch {
console.error("[BetterSEQTA+] Failed to fetch notices:", error); // Notices failed to load; processNotices will show empty state if container exists
processNotices({ payload: [] }, labelArray);
} }
}; };
@@ -308,19 +286,12 @@ function comparedate(obj1: any, obj2: any) {
} }
function processNotices(response: any, labelArray: string[]) { function processNotices(response: any, labelArray: string[]) {
const NoticeContainer = document.getElementById("notice-container"); const NoticeContainer = document.getElementById("notice-container");
if (!NoticeContainer) { if (!NoticeContainer) return;
console.warn("[BetterSEQTA+] Notices: processNotices — notice-container not found");
return;
}
NoticeContainer.innerHTML = ""; NoticeContainer.innerHTML = "";
const notices = response?.payload; const notices = response?.payload;
if (!Array.isArray(notices)) { if (!Array.isArray(notices)) {
console.warn("[BetterSEQTA+] Notices: processNotices — invalid or missing payload", {
hasResponse: !!response,
payloadType: typeof response?.payload,
});
const emptyState = document.createElement("div"); const emptyState = document.createElement("div");
emptyState.classList.add("day-empty"); emptyState.classList.add("day-empty");
const img = document.createElement("img"); const img = document.createElement("img");
@@ -333,7 +304,6 @@ function processNotices(response: any, labelArray: string[]) {
} }
if (!notices.length) { if (!notices.length) {
console.info("[BetterSEQTA+] Notices: processNotices — no notices for date");
const emptyState = document.createElement("div"); const emptyState = document.createElement("div");
emptyState.classList.add("day-empty"); emptyState.classList.add("day-empty");
const img = document.createElement("img"); const img = document.createElement("img");
@@ -346,7 +316,6 @@ function processNotices(response: any, labelArray: string[]) {
} }
const fragment = document.createDocumentFragment(); const fragment = document.createDocumentFragment();
let includedCount = 0;
notices.forEach((notice: any) => { notices.forEach((notice: any) => {
const shouldInclude = const shouldInclude =
@@ -354,17 +323,12 @@ function processNotices(response: any, labelArray: string[]) {
labelArray.includes(JSON.stringify(notice.label)); labelArray.includes(JSON.stringify(notice.label));
if (shouldInclude) { if (shouldInclude) {
includedCount++;
const colour = processNoticeColor(notice.colour); const colour = processNoticeColor(notice.colour);
const noticeElement = createNoticeElement(notice, colour); const noticeElement = createNoticeElement(notice, colour);
fragment.appendChild(noticeElement); fragment.appendChild(noticeElement);
} }
}); });
console.info("[BetterSEQTA+] Notices: processNotices — rendered", {
totalFromApi: notices.length,
includedAfterFilter: includedCount,
});
NoticeContainer.appendChild(fragment); NoticeContainer.appendChild(fragment);
} }