feat(dashboard): Startansicht mit Bewerbungen, Wandlung und Gewinnen

Erster Menuepunkt und Startansicht; das Logo oben links fuehrt dorthin zurueck. Vier Kennzahlen (Bewerbungen, daraus Opportunities samt Quote, gewonnene Projekte, aktuell beim Kunden), ein Saeulendiagramm mit denselben drei Reihen ueber der Zeit, die Bewerbungen je Berater, die naechsten Termine und alle Opportunities in Kundenvorstellung bzw. Interview Kunde. Zeitraum ueber vier Vorgaben oder frei, Einteilung nach Tag, Monat, Quartal oder Jahr. Bewerbung heisst dabei: ein Eintrag in der Akquise - jeder Eintrag bedeutet, dass ein Profil raus ist (so vom Nutzer festgelegt). Zwei Dinge dafuer am Datenmodell: - crm_opportunities.won_at haelt den Zeitpunkt des Wechsels nach "Beauftragt". Die Phase allein sagt nur, wie es jetzt steht, nicht wann es passiert ist. Beim Wechsel heraus wird das Datum geleert, sonst zaehlt das Dashboard einen Gewinn, den es nicht gibt. - Die Akquise loescht nach 90 Tagen - damit waere jede Jahresauswertung falsch. Der Aufraeumlauf sichert deshalb vorher Tag, Berater und Anzahl in crm_acquisition_history, in derselben Transaktion wie das Loeschen. Gesichert wird nur, was verschwindet; die Auswertung addiert beide Quellen und kann sich damit nicht ueberschneiden. Projektdaten sind nach dem Loeschen weiterhin weg. Termine und die Phasenlisten kommen nicht aus einem eigenen Endpunkt: Termine und Opportunities laedt die App ohnehin komplett, das Dashboard filtert sie selbst. Diagramme von Hand (SVG-freie Balken plus Tailwind-Uebergaenge) statt einer Bibliothek - drei Darstellungen rechtfertigen keine weitere Abhaengigkeit, und so kommen die Farben aus denselben Tokens wie der Rest. Die Zahlen laufen hoch, die Saeulen wachsen; wer Bewegung im System abgeschaltet hat, bekommt sofort den Endwert. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-08-18 15:02:35 +02:00
parent 42639c65fa
commit 6f57483f33
17 changed files with 1133 additions and 19 deletions
@@ -175,16 +175,50 @@ async function purgeOldEntries() {
  if (Date.now() - lastPurgeAt < 3_600_000) return;
  lastPurgeAt = Date.now();

-  const [result] = await pool.query(
-    `DELETE FROM crm_acquisitions
-      WHERE converted_at IS NULL
-        AND found_at < DATE_SUB(CURDATE(), INTERVAL ? DAY)`,
-    [DELETE_AFTER_DAYS]
-  );
-  if (result.affectedRows > 0) {
-    console.log(
-      `[Akquise] ${result.affectedRows} Eintrag/Eintraege aelter als ${DELETE_AFTER_DAYS} Tage geloescht.`
+  /*
+   * Vor dem Loeschen die Zahlen sichern: jeder Eintrag ist eine versendete
+   * Bewerbung, und das Dashboard rechnet ueber Jahre. Gesichert wird nur Tag,
+   * Berater und Anzahl - die Projektdaten sind danach wirklich weg.
+   *
+   * Sichern und Loeschen gehoeren in eine Transaktion: die Anzahl wird
+   * aufaddiert (ein Tag kann aus einem frueheren Lauf schon in der Historie
+   * stehen), und ein Abbruch zwischen beiden Schritten wuerde denselben Tag beim
+   * naechsten Lauf ein zweites Mal dazurechnen.
+   */
+  const conn = await pool.getConnection();
+  try {
+    await conn.beginTransaction();
+    await conn.query(
+      `INSERT INTO crm_acquisition_history (found_on, consultant, applications)
+       SELECT found_at, COALESCE(consultant, ''), COUNT(*)
+         FROM crm_acquisitions
+        WHERE converted_at IS NULL
+          AND found_at < DATE_SUB(CURDATE(), INTERVAL ? DAY)
+        GROUP BY found_at, COALESCE(consultant, '')
+       ON DUPLICATE KEY UPDATE
+         applications = crm_acquisition_history.applications + VALUES(applications)`,
+      [DELETE_AFTER_DAYS]
    );
+
+    const [result] = await conn.query(
+      `DELETE FROM crm_acquisitions
+        WHERE converted_at IS NULL
+          AND found_at < DATE_SUB(CURDATE(), INTERVAL ? DAY)`,
+      [DELETE_AFTER_DAYS]
+    );
+    await conn.commit();
+
+    if (result.affectedRows > 0) {
+      console.log(
+        `[Akquise] ${result.affectedRows} Eintrag/Eintraege aelter als ${DELETE_AFTER_DAYS} Tage geloescht (Anzahl im Verlauf gesichert).`
+      );
+    }
+  } catch (err) {
+    await conn.rollback().catch(() => {});
+    // Nicht weiterwerfen: das Aufraeumen darf die Liste nicht blockieren.
+    console.warn(`[Akquise] Aufraeumen fehlgeschlagen: ${err.message}`);
+  } finally {
+    conn.release();
  }
 }

@@ -0,0 +1,143 @@
+import express from 'express';
+import { pool } from '../db.js';
+import { requireAuth } from '../middleware/auth.js';
+
+const router = express.Router();
+router.use(requireAuth);
+
+/*
+ * Zahlen fuer das Dashboard. Drei Reihen ueber der Zeit:
+ *
+ *   Bewerbungen - Eintraege in der Akquise. Jeder Eintrag ist eine versendete
+ *                 Bewerbung (so vom Nutzer festgelegt), gezaehlt nach `found_at`.
+ *   Gewandelt   - daraus wurde eine Opportunity, gezaehlt nach `converted_at`.
+ *   Gewonnen    - Opportunity in der Phase "Beauftragt", gezaehlt nach `won_at`.
+ *
+ * Die Akquise loescht Eintraege nach 90 Tagen, sichert vorher aber Tag, Berater
+ * und Anzahl in crm_acquisition_history. Die Bewerbungsreihe liest deshalb aus
+ * beiden Quellen - lebende Eintraege und Verlauf - und zaehlt sie zusammen.
+ * Ueberschneiden koennen sie sich nicht: gesichert wird genau, was verschwindet.
+ *
+ * Termine und die Listen zu den Phasen stehen absichtlich nicht hier: Termine und
+ * Opportunities laedt die App ohnehin komplett, das Dashboard filtert sie selbst.
+ * Ein zweiter Weg zu denselben Daten waere doppelte Pflege.
+ */
+
+/** Wie die Zeitachse geschnitten wird. Der Ausdruck geht in GROUP BY. */
+const BUCKETS = {
+  tag: (column) => `DATE_FORMAT(${column}, '%Y-%m-%d')`,
+  monat: (column) => `DATE_FORMAT(${column}, '%Y-%m')`,
+  quartal: (column) => `CONCAT(YEAR(${column}), '-Q', QUARTER(${column}))`,
+  jahr: (column) => `YEAR(${column})`,
+};
+
+/** `YYYY-MM-DD` oder null. */
+function normalizeDate(value) {
+  const match = /^(\d{4})-(\d{2})-(\d{2})$/.exec(String(value ?? '').trim());
+  if (!match) return null;
+  const probe = new Date(`${match[0]}T00:00:00Z`);
+  return Number.isNaN(probe.getTime()) ? null : match[0];
+}
+
+/** Zeilen [{ bucket, count }] als Objekt - das Frontend legt die Achse selbst. */
+const toMap = (rows) =>
+  Object.fromEntries(rows.map((row) => [String(row.bucket), Number(row.count)]));
+
+// GET /api/dashboard?from=2026-01-01&to=2026-12-31&bucket=monat
+router.get('/', async (req, res, next) => {
+  const bucket = String(req.query.bucket ?? 'monat');
+  if (!BUCKETS[bucket]) return res.status(400).json({ error: 'Unbekannte Zeiteinteilung.' });
+
+  const from = normalizeDate(req.query.from);
+  const to = normalizeDate(req.query.to);
+  if (!from || !to) return res.status(400).json({ error: 'Zeitraum fehlt (erwartet JJJJ-MM-TT).' });
+  if (to < from) return res.status(400).json({ error: 'Das Ende darf nicht vor dem Anfang liegen.' });
+
+  // `to` ist inklusive: bis 31.12. heisst bis zum Ende des 31.12.
+  const toExclusive = `${to} 23:59:59`;
+
+  try {
+    const [applicationRows] = await pool.query(
+      `SELECT ${BUCKETS[bucket]('found_at')} AS bucket, COUNT(*) AS count
+         FROM crm_acquisitions
+        WHERE found_at BETWEEN ? AND ?
+        GROUP BY bucket`,
+      [from, to]
+    );
+
+    const [historyRows] = await pool.query(
+      `SELECT ${BUCKETS[bucket]('found_on')} AS bucket, SUM(applications) AS count
+         FROM crm_acquisition_history
+        WHERE found_on BETWEEN ? AND ?
+        GROUP BY bucket`,
+      [from, to]
+    );
+
+    const [convertedRows] = await pool.query(
+      `SELECT ${BUCKETS[bucket]('converted_at')} AS bucket, COUNT(*) AS count
+         FROM crm_acquisitions
+        WHERE converted_at IS NOT NULL AND converted_at BETWEEN ? AND ?
+        GROUP BY bucket`,
+      [from, toExclusive]
+    );
+
+    const [wonRows] = await pool.query(
+      `SELECT ${BUCKETS[bucket]('won_at')} AS bucket, COUNT(*) AS count
+         FROM crm_opportunities
+        WHERE won_at IS NOT NULL AND deleted_at IS NULL AND won_at BETWEEN ? AND ?
+        GROUP BY bucket`,
+      [from, toExclusive]
+    );
+
+    // Bewerbungen je Berater - lebende Eintraege und Verlauf zusammen.
+    const [consultantRows] = await pool.query(
+      `SELECT consultant, SUM(count) AS count FROM (
+         SELECT COALESCE(NULLIF(TRIM(consultant), ''), '') AS consultant, COUNT(*) AS count
+           FROM crm_acquisitions
+          WHERE found_at BETWEEN ? AND ?
+          GROUP BY 1
+         UNION ALL
+         SELECT consultant, SUM(applications)
+           FROM crm_acquisition_history
+          WHERE found_on BETWEEN ? AND ?
+          GROUP BY 1
+       ) AS beide
+       GROUP BY consultant
+       ORDER BY count DESC, consultant ASC`,
+      [from, to, from, to]
+    );
+
+    const live = toMap(applicationRows);
+    const history = toMap(historyRows);
+    const applications = { ...live };
+    for (const [key, value] of Object.entries(history)) {
+      applications[key] = (applications[key] ?? 0) + value;
+    }
+
+    const sum = (map) => Object.values(map).reduce((total, value) => total + value, 0);
+    const converted = toMap(convertedRows);
+    const won = toMap(wonRows);
+
+    res.json({
+      from,
+      to,
+      bucket,
+      series: { applications, converted, won },
+      totals: {
+        applications: sum(applications),
+        converted: sum(converted),
+        won: sum(won),
+      },
+      by_consultant: consultantRows.map((row) => ({
+        // Leerer Text heisst "kein Berater hinterlegt" - das Frontend
+        // beschriftet es, damit hier kein Anzeigetext entsteht.
+        consultant: row.consultant === '' ? null : row.consultant,
+        applications: Number(row.count),
+      })),
+    });
+  } catch (err) {
+    next(err);
+  }
+});
+
+export default router;
@@ -310,11 +310,13 @@ router.post('/', async (req, res, next) => {
  try {
    await conn.beginTransaction();
    const [result] = await conn.query(
+      // Wer direkt in 'Beauftragt' anlegt, hat gerade gewonnen - dann steht das
+      // Datum sofort.
      `INSERT INTO crm_opportunities
-         (title, company, description, stage, probability, daily_rate, duration_days,
+         (title, company, description, stage, won_at, probability, daily_rate, duration_days,
          fixed_price, start_date, end_date, last_interaction_at, last_interaction_note,
          next_step, tags, notes, owner_id, created_by)
-       VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,
+       VALUES (?, ?, ?, ?, ${stage === 'Beauftragt' ? 'UTC_TIMESTAMP()' : 'NULL'}, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,
      [
        title,
        emptyToNull(req.body?.company),
@@ -387,6 +389,18 @@ router.patch('/:id', async (req, res, next) => {
      if (!STAGES.has(req.body.stage)) return res.status(400).json({ error: 'Ungültige Phase.' });
      fields.push('stage = ?');
      values.push(req.body.stage);
+
+      /*
+       * Zeitpunkt des Gewinns mitschreiben. Der erste Wechsel nach 'Beauftragt'
+       * zaehlt; ein zweites Speichern derselben Phase darf das Datum nicht nach
+       * vorn schieben. Geht die Opportunity wieder heraus, wird es geleert -
+       * sonst zaehlt das Dashboard einen Gewinn, den es nicht gibt.
+       */
+      if (req.body.stage === 'Beauftragt' && existing.stage !== 'Beauftragt') {
+        fields.push('won_at = UTC_TIMESTAMP()');
+      } else if (req.body.stage !== 'Beauftragt' && existing.stage === 'Beauftragt') {
+        fields.push('won_at = NULL');
+      }
    }

    if (req.body?.probability !== undefined) {
@@ -158,6 +158,22 @@ ALTER TABLE crm_opportunities ADD COLUMN IF NOT EXISTS fixed_price DECIMAL(12,2)
 * `deleted_by` traegt bewusst keinen Fremdschluessel: Spalten lassen sich per
 * ADD COLUMN IF NOT EXISTS idempotent nachziehen, ein Fremdschluessel nicht.
 */
+/*
+ * Zeitpunkt des Gewinns. Ohne ihn liesse sich "wie viele Projekte haben wir im
+ * Mai gewonnen" nicht beantworten - die Phase allein sagt nur, wie es jetzt
+ * steht. Gesetzt beim Wechsel nach 'Beauftragt', geleert beim Wechsel heraus.
+ */
+ALTER TABLE crm_opportunities ADD COLUMN IF NOT EXISTS won_at DATETIME NULL AFTER stage;
+ALTER TABLE crm_opportunities ADD INDEX IF NOT EXISTS idx_crm_opp_won (won_at);
+
+/*
+ * Zeitpunkt des Gewinns. Ohne ihn liesse sich "wie viele Projekte haben wir im
+ * Mai gewonnen" nicht beantworten - die Phase allein sagt nur, wie es jetzt
+ * steht. Gesetzt beim Wechsel nach 'Beauftragt', geleert beim Wechsel heraus.
+ */
+ALTER TABLE crm_opportunities ADD COLUMN IF NOT EXISTS won_at DATETIME NULL AFTER stage;
+ALTER TABLE crm_opportunities ADD INDEX IF NOT EXISTS idx_crm_opp_won (won_at);
+
 ALTER TABLE crm_opportunities ADD COLUMN IF NOT EXISTS deleted_at DATETIME NULL AFTER created_by;
 ALTER TABLE crm_opportunities ADD COLUMN IF NOT EXISTS deleted_by INT UNSIGNED NULL AFTER deleted_at;
 ALTER TABLE crm_opportunities ADD INDEX IF NOT EXISTS idx_crm_opp_deleted (deleted_at);
@@ -432,6 +448,40 @@ CREATE TABLE IF NOT EXISTS crm_acquisitions (
 * Der Unique-Schluessel greift ueber die Kollation utf8mb4_unicode_ci auch bei
 * abweichender Gross-/Kleinschreibung: "mail" ist neben "Mail" kein neuer Wert.
 */
+/*
+ * Verdichteter Verlauf der Akquise. Jeder Eintrag in crm_acquisitions ist eine
+ * versendete Bewerbung; nach 90 Tagen wird er geloescht (siehe dort). Damit das
+ * Dashboard trotzdem ueber Jahre rechnen kann, wird vor dem Loeschen genau das
+ * gesichert, was die Auswertung braucht: Tag, Berater, Anzahl.
+ *
+ * Bewusst nur Zahlen, keine Projektdaten - "geloescht" bleibt damit auch
+ * geloescht. Der Primaerschluessel erlaubt kein NULL, deshalb steht fuer "ohne
+ * Berater" der leere Text.
+ */
+CREATE TABLE IF NOT EXISTS crm_acquisition_history (
+  found_on     DATE         NOT NULL,
+  consultant   VARCHAR(190) NOT NULL DEFAULT '',
+  applications INT UNSIGNED NOT NULL DEFAULT 0,
+  PRIMARY KEY (found_on, consultant)
+) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;
+
+/*
+ * Verdichteter Verlauf der Akquise. Jeder Eintrag in crm_acquisitions ist eine
+ * versendete Bewerbung; nach 90 Tagen wird er geloescht (siehe dort). Damit das
+ * Dashboard trotzdem ueber Jahre rechnen kann, wird vor dem Loeschen genau das
+ * gesichert, was die Auswertung braucht: Tag, Berater, Anzahl.
+ *
+ * Bewusst nur Zahlen, keine Projektdaten - "geloescht" bleibt damit auch
+ * geloescht. Der Primaerschluessel erlaubt kein NULL, deshalb steht fuer "ohne
+ * Berater" der leere Text.
+ */
+CREATE TABLE IF NOT EXISTS crm_acquisition_history (
+  found_on     DATE         NOT NULL,
+  consultant   VARCHAR(190) NOT NULL DEFAULT '',
+  applications INT UNSIGNED NOT NULL DEFAULT 0,
+  PRIMARY KEY (found_on, consultant)
+) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;
+
 CREATE TABLE IF NOT EXISTS crm_acquisition_options (
  id         INT UNSIGNED NOT NULL AUTO_INCREMENT,
  field      ENUM('sender','channel','consultant') NOT NULL,
@@ -10,6 +10,7 @@ import eventRoutes from './routes/events.js';
 import opportunityRoutes from './routes/opportunities.js';
 import acquisitionRoutes from './routes/acquisitions.js';
 import badgeRoutes from './routes/badges.js';
+import dashboardRoutes from './routes/dashboard.js';
 import scannerRoutes from './routes/scanner.js';
 import mailRoutes from './routes/mail.js';
 import { requireAuth } from './middleware/auth.js';
@@ -54,6 +55,9 @@ app.use('/api/acquisitions', acquisitionRoutes);
 // Zahlen fuer die Seitenleiste - eine Anfrage fuer alle Zaehler.
 app.use('/api/badges', badgeRoutes);

+// Auswertungen des Dashboards.
+app.use('/api/dashboard', dashboardRoutes);
+
 // Projekt Scanner. Komplett hinter der CRM-Anmeldung - der urspruengliche
 // HTTP-Basic-Auth-Zugang des Scanners entfaellt damit.
 app.use('/api/scanner', requireAuth, scannerRoutes);