From 2e7378710fb35eb6264f3b48d84a1722e30be452 Mon Sep 17 00:00:00 2001 From: Till-JS <101404291+Till-JS@users.noreply.github.com> Date: Wed, 28 Jan 2026 12:31:43 +0100 Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=B9=20chore(calendar):=20cleanup=20deb?= =?UTF-8?q?ug=20logs=20and=20stale=20comments?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Replace console.error with NestJS Logger in NetworkService - Remove debug console.log statements from mana page and events store - Remove stale TODO comment in ShareController (user.email already available) - Update TODO comments to "Workaround" notes in EventContextMenu - Fix port fallback in main.ts (3016 → 3014) for consistency --- apps/calendar/apps/backend/src/main.ts | 2 +- apps/calendar/apps/backend/src/network/network.service.ts | 7 ++++--- apps/calendar/apps/backend/src/share/share.controller.ts | 1 - .../web/src/lib/components/event/EventContextMenu.svelte | 4 ++-- apps/calendar/apps/web/src/lib/stores/events.svelte.ts | 1 - apps/calendar/apps/web/src/routes/(app)/mana/+page.svelte | 2 -- 6 files changed, 7 insertions(+), 10 deletions(-) diff --git a/apps/calendar/apps/backend/src/main.ts b/apps/calendar/apps/backend/src/main.ts index cca374b4e..7cf6f2ece 100644 --- a/apps/calendar/apps/backend/src/main.ts +++ b/apps/calendar/apps/backend/src/main.ts @@ -73,7 +73,7 @@ async function bootstrap() { exclude: ['metrics', 'health'], }); - const port = process.env.PORT || 3016; + const port = process.env.PORT || 3014; await app.listen(port); console.log(`Calendar backend running on http://localhost:${port}`); } diff --git a/apps/calendar/apps/backend/src/network/network.service.ts b/apps/calendar/apps/backend/src/network/network.service.ts index 1225187c6..83ce8e701 100644 --- a/apps/calendar/apps/backend/src/network/network.service.ts +++ b/apps/calendar/apps/backend/src/network/network.service.ts @@ -1,4 +1,4 @@ -import { Injectable, Inject } from '@nestjs/common'; +import { Injectable, Inject, Logger } from '@nestjs/common'; import { ConfigService } from '@nestjs/config'; import { eq } from 'drizzle-orm'; import { DATABASE_CONNECTION } from '../db/database.module'; @@ -36,6 +36,7 @@ export interface NetworkGraphResponse { @Injectable() export class NetworkService { + private readonly logger = new Logger(NetworkService.name); private authUrl: string; constructor( @@ -59,14 +60,14 @@ export class NetworkService { }); if (!response.ok) { - console.error('Failed to fetch tags from central API:', response.status); + this.logger.warn(`Failed to fetch tags from central API: ${response.status}`); return new Map(); } const tags: Tag[] = await response.json(); return new Map(tags.map((t) => [t.id, t])); } catch (error) { - console.error('Error fetching tags from central API:', error); + this.logger.error('Error fetching tags from central API', error); return new Map(); } } diff --git a/apps/calendar/apps/backend/src/share/share.controller.ts b/apps/calendar/apps/backend/src/share/share.controller.ts index 3ef418bb0..6b5ba3610 100644 --- a/apps/calendar/apps/backend/src/share/share.controller.ts +++ b/apps/calendar/apps/backend/src/share/share.controller.ts @@ -48,7 +48,6 @@ export class ShareController { @Get('shares/invitations') async getInvitations(@CurrentUser() user: CurrentUserData) { - // TODO: Get user email from auth service const invitations = await this.shareService.findPendingInvitations( user.userId, user.email || '' diff --git a/apps/calendar/apps/web/src/lib/components/event/EventContextMenu.svelte b/apps/calendar/apps/web/src/lib/components/event/EventContextMenu.svelte index a3b351229..0d7c36e2b 100644 --- a/apps/calendar/apps/web/src/lib/components/event/EventContextMenu.svelte +++ b/apps/calendar/apps/web/src/lib/components/event/EventContextMenu.svelte @@ -102,7 +102,7 @@ } function handleChangeCalendar() { - // TODO: Implement calendar picker modal + // Workaround: cycles through calendars until modal is implemented const event = eventContextMenuStore.targetEvent; if (!event) return; @@ -119,7 +119,7 @@ } function handleChangeColor() { - // TODO: Implement color picker modal + // Workaround: cycles through colors until modal is implemented const event = eventContextMenuStore.targetEvent; if (!event) return; diff --git a/apps/calendar/apps/web/src/lib/stores/events.svelte.ts b/apps/calendar/apps/web/src/lib/stores/events.svelte.ts index 6db5ee9ec..c19d67704 100644 --- a/apps/calendar/apps/web/src/lib/stores/events.svelte.ts +++ b/apps/calendar/apps/web/src/lib/stores/events.svelte.ts @@ -74,7 +74,6 @@ export const eventsStore = { } else { // API returns events array directly (already extracted in api/events.ts) const eventsData = result.data as CalendarEvent[] | null; - console.log('[Events Store] Loaded events:', eventsData?.length, eventsData); events = eventsData || []; loadedRange = { start: startDate, end: endDate }; } diff --git a/apps/calendar/apps/web/src/routes/(app)/mana/+page.svelte b/apps/calendar/apps/web/src/routes/(app)/mana/+page.svelte index fe14b49ab..68d2cff27 100644 --- a/apps/calendar/apps/web/src/routes/(app)/mana/+page.svelte +++ b/apps/calendar/apps/web/src/routes/(app)/mana/+page.svelte @@ -3,12 +3,10 @@ import { toast } from '$lib/stores/toast.svelte'; function handleSubscribe(planId: string) { - console.log('Subscribe to plan:', planId); toast.info(`Abo "${planId}" ausgewählt. Bezahlsystem wird noch integriert.`); } function handleBuyPackage(packageId: string) { - console.log('Buy package:', packageId); toast.info(`Paket "${packageId}" ausgewählt. Bezahlsystem wird noch integriert.`); }