@@ -17,6 +17,18 @@ und dieses Projekt folgt [Semantic Versioning](https://semver.org/lang/de/).
|
||||
|
||||
### Sicherheit
|
||||
|
||||
## [0.21.0] – 2026-10-03
|
||||
|
||||
### Hinzugefügt
|
||||
- Standort eines Hosts lässt sich in der Detailansicht setzen; die Auswahl richtet sich nach dem gewählten Kunden (#036)
|
||||
- Breite Tabellen werden auf dem Telefon zu Karten: jede Angabe mit ihrer Beschriftung, statt rechts abgeschnitten zu sein (#035)
|
||||
|
||||
### Behoben
|
||||
- **Jede Kundenzuweisung löschte den Standort des Hosts**, weil das Formular ihn nicht mitsendete, die Aktion ihn aber las (#036)
|
||||
- Ein Standort, der zu einem anderen Kunden gehört, wird nicht mehr gespeichert (#036)
|
||||
- Kartenköpfe mit Knöpfen liefen auf dem Telefon über den Rand (#035)
|
||||
- Das Kontrollkästchen für den Rechnungsstatus wurde auf dem Telefon zum 44 Pixel hohen leeren Kasten (#035)
|
||||
|
||||
## [0.20.2] – 2026-10-03
|
||||
|
||||
### Behoben
|
||||
|
||||
@@ -2,7 +2,7 @@
|
||||
|
||||
## Zeitübersicht
|
||||
|
||||
**Gesamt:** 17 h 38 min
|
||||
**Gesamt:** 18 h 58 min
|
||||
|
||||
| Issue | Zeit |
|
||||
|---|---|
|
||||
@@ -40,6 +40,8 @@
|
||||
| #032 LXC-Installation korrigiert | 0 h 20 min |
|
||||
| #033 Weiterleitung auf Lausch-Adresse | 0 h 30 min |
|
||||
| #034 Bedienung auf dem Handy | 0 h 40 min |
|
||||
| #035 Tabellen als Karten | 0 h 50 min |
|
||||
| #036 Standort am Host | 0 h 30 min |
|
||||
|
||||
| Meilenstein | Zeit |
|
||||
|---|---|
|
||||
@@ -62,7 +64,7 @@
|
||||
|
||||
## 2026-09-17 · Sitzung 1 · 19:36–laufend
|
||||
**Issue:** #001–#007 (Meilensteine v0.1.0 bis v0.4.0)
|
||||
**Version:** 0.20.2
|
||||
**Version:** 0.21.0
|
||||
|
||||
### Erledigt
|
||||
- Auftrag gelesen, Rückfragen gestellt
|
||||
@@ -183,6 +185,10 @@
|
||||
- Mobile Bedienung (#034): Das Menü ließ sich auf dem Telefon nicht schließen – AdminLTE-Markup ohne AdminLTE-JavaScript. Eigener Zustand, abdunkelnde Fläche, Schließen beim Navigieren und mit Escape
|
||||
- Lehre: Die Mobilansicht war mit Screenshots geprüft, nicht mit den Fingern. Ein Bild zeigt nicht, dass etwas nicht wieder weggeht
|
||||
|
||||
- Breite Tabellen werden auf dem Telefon zu Karten (#035); dabei fiel auf, dass `pl-touch-target` am Kontrollkästchen selbst saß und es zum leeren Kasten aufblies
|
||||
- Standort am Host (#036): Die Meldung lautete „kann man nicht einstellen“ – tatsächlich löschte jede Kundenzuweisung den vorhandenen Standort, weil das Formular ihn nicht mitsendete und die Aktion ihn trotzdem las
|
||||
- Beide Fehler kamen aus dem echten Einsatz, nicht aus den Tests. Das bleibt die ergiebigste Quelle
|
||||
|
||||
### Nächster Schritt
|
||||
- #016 Benutzerhandbuch mit Screenshots, danach #017 Favicons und #019 CI-Pipeline
|
||||
- Für 1.0.0 fehlt noch: PMG (kein Host verfügbar) und der Fall „aktive Subscription auf PBS“ (#023)
|
||||
|
||||
@@ -175,6 +175,9 @@ weitergeben, und der Zurück-Knopf funktioniert wie erwartet.
|
||||
|
||||
---
|
||||
|
||||
Im Detail eines Hosts lassen sich Kunde **und Standort** gemeinsam ändern. Die Standortliste zeigt nur
|
||||
die Standorte des gewählten Kunden; angelegt werden sie auf der Kundenseite.
|
||||
|
||||
## 4a. Cluster
|
||||
|
||||
**In Proxmox hat jeder Knoten eine eigene Subscription** mit eigener Server-ID – ein Cluster mit drei
|
||||
|
||||
|
Before Width: | Height: | Size: 131 KiB After Width: | Height: | Size: 131 KiB |
|
Before Width: | Height: | Size: 114 KiB After Width: | Height: | Size: 114 KiB |
|
Before Width: | Height: | Size: 195 KiB After Width: | Height: | Size: 196 KiB |
|
Before Width: | Height: | Size: 121 KiB After Width: | Height: | Size: 121 KiB |
|
Before Width: | Height: | Size: 121 KiB After Width: | Height: | Size: 120 KiB |
|
After Width: | Height: | Size: 109 KiB |
|
After Width: | Height: | Size: 86 KiB |
|
After Width: | Height: | Size: 97 KiB |
|
Before Width: | Height: | Size: 139 KiB After Width: | Height: | Size: 139 KiB |
|
Before Width: | Height: | Size: 103 KiB After Width: | Height: | Size: 103 KiB |
|
Before Width: | Height: | Size: 169 KiB After Width: | Height: | Size: 170 KiB |
|
Before Width: | Height: | Size: 169 KiB After Width: | Height: | Size: 170 KiB |
|
Before Width: | Height: | Size: 178 KiB After Width: | Height: | Size: 178 KiB |
@@ -0,0 +1,28 @@
|
||||
# #035 Tabellen auf dem Handy als Karten
|
||||
**Status:** erledigt
|
||||
**Meilenstein:** v0.21.0
|
||||
**Labels:** ui
|
||||
**Geschätzt:** 3 h **Tatsächlich:** 0 h 50 min
|
||||
|
||||
## Auslöser
|
||||
„Etwas schöner darstellen auf dem Handy muss doch drin sein.“ Auf Rechnungen, Kunden und Jahresbahn
|
||||
war rechts jeweils die Hälfte abgeschnitten, Knöpfe im Kartenkopf liefen über den Rand, und Namen
|
||||
brachen Buchstabe für Buchstabe um.
|
||||
|
||||
## Aufgaben
|
||||
- [x] `.pl-karten-tabelle`: unterhalb von 768 px wird jede Tabellenzeile zur Karte, jede Zelle bekommt
|
||||
ihre Beschriftung aus `data-label` davor
|
||||
- [x] Angewandt auf Kunden, Rechnungen und den Ablauf-Kalender der Jahresbahn
|
||||
- [x] Kartenkopf bricht um, statt über den Rand zu laufen
|
||||
- [x] Jahresbahn: schmalere Namensspalte auf dem Telefon, dazu der Hinweis aufs seitliche Wischen
|
||||
- [x] Mobilansichten der breiten Seiten kommen jetzt mit in die Bilderstrecke
|
||||
|
||||
## Warum Karten statt Scrollen
|
||||
Eine Tabelle mit elf Spalten lässt sich auf 390 px nicht sinnvoll anzeigen. Seitliches Scrollen
|
||||
versteckt die Hälfte der Angaben, und Spalten wegzulassen nimmt sie ganz weg. Als Karte steht jede
|
||||
Angabe mit ihrer Beschriftung da – länger, aber vollständig.
|
||||
|
||||
## Nebenbefund
|
||||
Das Kontrollkästchen für den Rechnungsstatus trug `pl-touch-target`, also `min-height: 44px` – und
|
||||
wurde damit selbst zum 44 Pixel hohen leeren Kasten. Die Fläche zum Treffen bekommt jetzt sein Umfeld,
|
||||
das Kästchen behält seine Größe.
|
||||
@@ -0,0 +1,22 @@
|
||||
# #036 Standort am Host setzen
|
||||
**Status:** erledigt
|
||||
**Meilenstein:** v0.21.0
|
||||
**Labels:** bug, ui
|
||||
**Geschätzt:** 1 h **Tatsächlich:** 0 h 30 min
|
||||
|
||||
## Auslöser
|
||||
„Den Standort bei den Hosts kann man nicht einstellen.“
|
||||
|
||||
## Was wirklich los war
|
||||
Schlimmer als gemeldet: Das Formular in der Host-Detailansicht sendete **nur** den Kunden. Die Aktion
|
||||
`hostKundeSetzen` las aber auch `standortId` – und bekam nichts. Jede Kundenzuweisung hat damit den
|
||||
Standort **stillschweigend gelöscht**. Setzen ging nur beim Ausrollen über die Sammelzuweisung unter
|
||||
*Hosts → Nicht zugeordnet*; danach war er beim nächsten Speichern wieder weg.
|
||||
|
||||
## Behebung
|
||||
- Kunde und Standort in **einem** Formular, die Standortliste richtet sich nach dem gewählten Kunden
|
||||
- Wechselt der Kunde, wird die Standortauswahl zurückgesetzt statt stehen zu bleiben
|
||||
- Hat ein Kunde keine Standorte, sagt das Formular es, statt eine leere Auswahl anzubieten
|
||||
- Die Aktion prüft jetzt, dass der Standort zum Kunden gehört, und verwirft ihn sonst – eine falsche
|
||||
Zuordnung wäre schlimmer als keine
|
||||
- Vier Tests, darunter der Fall „Standort eines anderen Kunden“
|
||||
@@ -38,4 +38,6 @@ Details und Architektur: [../architektur.md](../architektur.md). Zeiten werden a
|
||||
| [#032](032-lxc-installation-korrekturen.md) | LXC-Installation: Herkunft der Skripte und Speicher | erledigt | v0.20.1 | 1 h | 0 h 20 min |
|
||||
| [#033](033-weiterleitung-lauschadresse.md) | Abmelden führte auf die Lausch-Adresse | erledigt | v0.20.1 | 1 h | 0 h 30 min |
|
||||
| [#034](034-mobile-bedienung.md) | Bedienung auf dem Handy | erledigt | v0.20.2 | 2 h | 0 h 40 min |
|
||||
| | **Summe** | | | **153 h** | **17 h 38 min** |
|
||||
| [#035](035-mobile-tabellen.md) | Tabellen auf dem Handy als Karten | erledigt | v0.21.0 | 3 h | 0 h 50 min |
|
||||
| [#036](036-standort-am-host.md) | Standort am Host setzen | erledigt | v0.21.0 | 1 h | 0 h 30 min |
|
||||
| | **Summe** | | | **157 h** | **18 h 58 min** |
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "proxledger",
|
||||
"version": "0.20.2",
|
||||
"version": "0.21.0",
|
||||
"private": true,
|
||||
"scripts": {
|
||||
"dev": "next dev",
|
||||
|
||||
@@ -195,7 +195,8 @@ async function main() {
|
||||
console.log(` ${seite}-dunkel.png`);
|
||||
}
|
||||
|
||||
for (const seite of ["dashboard", "hosts"]) {
|
||||
// Die Seiten mit den breitesten Tabellen – dort entscheidet sich die Mobilansicht (#035).
|
||||
for (const seite of ["dashboard", "hosts", "kunden", "rechnungen", "jahresbahn"]) {
|
||||
const eintrag = SEITEN.find((s) => s.name === seite);
|
||||
await aufnehmen(hell, eintrag.pfad, path.join(ZIEL, `mobil-${seite}.png`), 390, 1500, adresse);
|
||||
console.log(` mobil-${seite}.png`);
|
||||
|
||||
@@ -16,7 +16,7 @@ import { Tabs } from "@/components/ui/Tabs";
|
||||
import { PRODUKT_TEXT } from "@/lib/anzeige";
|
||||
import { prisma } from "@/lib/db";
|
||||
import { formatDatum, formatDatumZeit, formatEuro, formatUptime, formatZahl } from "@/lib/format";
|
||||
import { hostKundeSetzen } from "@/server/actions/host-detail";
|
||||
import { KundeStandortFormular } from "@/components/hosts/KundeStandortFormular";
|
||||
import { hostTokenWiderrufen } from "@/server/actions/tokens";
|
||||
import { hostMitDetails } from "@/server/queries/hostliste";
|
||||
import { meldungsGrenze } from "@/server/queries/hosts";
|
||||
@@ -43,7 +43,11 @@ export default async function HostDetailSeite({ params }: { params: Promise<{ id
|
||||
|
||||
const kunden = await prisma.kunde.findMany({
|
||||
orderBy: { name: "asc" },
|
||||
select: { id: true, name: true },
|
||||
select: {
|
||||
id: true,
|
||||
name: true,
|
||||
standorte: { orderBy: { name: "asc" }, select: { id: true, name: true } },
|
||||
},
|
||||
});
|
||||
|
||||
// Welche Cluster-Mitglieder melden sich selbst? Fehlende Knoten haben keinen Agent (#022).
|
||||
@@ -167,30 +171,12 @@ export default async function HostDetailSeite({ params }: { params: Promise<{ id
|
||||
<Zeile bezeichnung="Erstmals gesehen">{formatDatum(host.erstmalsGesehenAm)}</Zeile>
|
||||
<Zeile bezeichnung="Letzte Meldung">{formatDatumZeit(host.letzterReportAm)}</Zeile>
|
||||
|
||||
<form action={hostKundeSetzen} className="mt-3">
|
||||
<input type="hidden" name="hostId" value={host.id} />
|
||||
<label className="form-label small" htmlFor="kundeId">
|
||||
Kunde ändern
|
||||
</label>
|
||||
<div className="input-group input-group-sm">
|
||||
<select
|
||||
id="kundeId"
|
||||
name="kundeId"
|
||||
className="form-select"
|
||||
defaultValue={host.kundeId ?? ""}
|
||||
>
|
||||
<option value="">– nicht zugeordnet –</option>
|
||||
{kunden.map((kunde) => (
|
||||
<option key={kunde.id} value={kunde.id}>
|
||||
{kunde.name}
|
||||
</option>
|
||||
))}
|
||||
</select>
|
||||
<button type="submit" className="btn btn-outline-primary">
|
||||
Setzen
|
||||
</button>
|
||||
</div>
|
||||
</form>
|
||||
<KundeStandortFormular
|
||||
hostId={host.id}
|
||||
kunden={kunden}
|
||||
kundeId={host.kundeId}
|
||||
standortId={host.standortId}
|
||||
/>
|
||||
</Card>
|
||||
</div>
|
||||
|
||||
|
||||
@@ -41,8 +41,8 @@ export default async function JahresbahnSeite() {
|
||||
<p className="mb-0 text-body-secondary">Noch kein Host mit Ablaufdatum gemeldet.</p>
|
||||
) : (
|
||||
<div className="table-responsive">
|
||||
<div style={{ minWidth: "820px" }}>
|
||||
<div className="d-flex text-body-secondary small mb-2" style={{ marginLeft: "13rem" }}>
|
||||
<div className="pl-timeline">
|
||||
<div className="d-flex text-body-secondary small mb-2 pl-timeline-kopf">
|
||||
{monate.map((monat) => (
|
||||
<div key={monat} style={{ width: `${100 / 12}%` }}>
|
||||
{monat}
|
||||
@@ -72,7 +72,7 @@ export default async function JahresbahnSeite() {
|
||||
: "var(--pl-info)";
|
||||
return (
|
||||
<div key={eintrag.hostId} className="d-flex align-items-center mb-1">
|
||||
<div className="small text-truncate" style={{ width: "13rem" }}>
|
||||
<div className="small text-truncate pl-timeline-name">
|
||||
<Link href={`/hosts/${eintrag.hostId}`} className="text-decoration-none">
|
||||
{eintrag.hostname}
|
||||
</Link>
|
||||
@@ -112,7 +112,8 @@ export default async function JahresbahnSeite() {
|
||||
)}
|
||||
|
||||
<p className="small text-body-secondary mb-0">
|
||||
Die rote Linie markiert heute ({formatDatum(jetzt)}). Balkenfarbe: rot unter 14 Tagen, gelb
|
||||
<span className="d-md-none">Seitwärts wischen zeigt die späteren Monate. </span>Die rote
|
||||
Linie markiert heute ({formatDatum(jetzt)}). Balkenfarbe: rot unter 14 Tagen, gelb
|
||||
unter 30 Tagen, sonst blau. Ein Klick führt zur Laufzeit mit ihren Rechnungen.
|
||||
</p>
|
||||
</Card>
|
||||
@@ -125,11 +126,11 @@ export default async function JahresbahnSeite() {
|
||||
tabelle
|
||||
>
|
||||
<div className="table-responsive">
|
||||
<table className="table table-striped align-middle">
|
||||
<table className="table table-striped align-middle pl-karten-tabelle">
|
||||
<thead>
|
||||
<tr>
|
||||
<th scope="col">Ablauf</th>
|
||||
<th scope="col">Host</th>
|
||||
<th scope="col">Ablauf</th>
|
||||
<th scope="col">Kunde</th>
|
||||
<th scope="col">Key</th>
|
||||
<th scope="col">Proxmox</th>
|
||||
@@ -139,15 +140,15 @@ export default async function JahresbahnSeite() {
|
||||
<tbody>
|
||||
{naechste90Tage.map((eintrag) => (
|
||||
<tr key={eintrag.hostId}>
|
||||
<td>
|
||||
<AblaufBadge datum={eintrag.ablauf.toISOString()} heute={jetzt} />
|
||||
</td>
|
||||
<td>
|
||||
<Link href={`/hosts/${eintrag.hostId}`} className="text-decoration-none">
|
||||
<td className="pl-karten-kopf">
|
||||
<Link href={`/hosts/${eintrag.hostId}`} className="fw-semibold text-decoration-none">
|
||||
{eintrag.hostname}
|
||||
</Link>
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Ablauf">
|
||||
<AblaufBadge datum={eintrag.ablauf.toISOString()} heute={jetzt} />
|
||||
</td>
|
||||
<td data-label="Kunde">
|
||||
{eintrag.kunde ? (
|
||||
<Link href={`/kunden/${eintrag.kunde.id}`} className="text-decoration-none">
|
||||
{eintrag.kunde.name}
|
||||
@@ -156,13 +157,13 @@ export default async function JahresbahnSeite() {
|
||||
<span className="badge pl-bg-neutral">Nicht zugeordnet</span>
|
||||
)}
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Key">
|
||||
<CopyableCode wert={eintrag.key} />
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Proxmox">
|
||||
<EinkaufBadge bezahlt={eintrag.einkaufBezahlt} />
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Kundenrechnung">
|
||||
<VerkaufBadge erstellt={eintrag.verkaufErstellt ?? false} bezahlt={false} />
|
||||
</td>
|
||||
</tr>
|
||||
|
||||
@@ -39,7 +39,7 @@ export default async function KundenSeite() {
|
||||
}
|
||||
>
|
||||
<div className="table-responsive">
|
||||
<table className="table table-striped align-middle">
|
||||
<table className="table table-striped align-middle pl-karten-tabelle">
|
||||
<thead>
|
||||
<tr>
|
||||
<th scope="col">Kunde</th>
|
||||
@@ -56,26 +56,28 @@ export default async function KundenSeite() {
|
||||
const punkte = rechnungspunkte.get(kunde.id);
|
||||
return (
|
||||
<tr key={kunde.id}>
|
||||
<td>
|
||||
<td className="pl-karten-kopf">
|
||||
<Link href={`/kunden/${kunde.id}`} className="fw-semibold text-decoration-none">
|
||||
{kunde.name}
|
||||
</Link>
|
||||
<div className="small text-body-secondary">{kunde.ansprechpartner ?? "–"}</div>
|
||||
</td>
|
||||
<td className="pl-mono">{kunde.kundennummer}</td>
|
||||
<td>{kunde.hosts}</td>
|
||||
<td>{kunde.aktiv}</td>
|
||||
<td>
|
||||
<td className="pl-mono" data-label="Kundennummer">
|
||||
{kunde.kundennummer}
|
||||
</td>
|
||||
<td data-label="Hosts">{kunde.hosts}</td>
|
||||
<td data-label="Aktiv lizenziert">{kunde.aktiv}</td>
|
||||
<td data-label="Nächster Ablauf">
|
||||
<AblaufBadge datum={kunde.naechsterAblauf?.toISOString() ?? null} heute={heute} />
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Warnungen">
|
||||
{kunde.offeneAlerts > 0 ? (
|
||||
<Badge farbe="warning">{kunde.offeneAlerts} offen</Badge>
|
||||
) : (
|
||||
<Badge farbe="success">keine</Badge>
|
||||
)}
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Rechnungen">
|
||||
{punkte && punkte.offen > 0 ? (
|
||||
<Badge farbe="danger">{punkte.offen} offen</Badge>
|
||||
) : (
|
||||
|
||||
@@ -197,7 +197,7 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
</form>
|
||||
|
||||
<div className="table-responsive">
|
||||
<table className="table table-striped align-middle">
|
||||
<table className="table table-striped align-middle pl-karten-tabelle">
|
||||
<thead>
|
||||
<tr>
|
||||
<th scope="col">Kunde</th>
|
||||
@@ -234,7 +234,13 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
const faelligIn = tageBis(zeile.einkaufFaelligAm, heute);
|
||||
return (
|
||||
<tr key={zeile.id}>
|
||||
<td>
|
||||
{/* Auf dem Telefon steht der Host als Überschrift der Karte, sonst als Spalte. */}
|
||||
<td className="d-md-none pl-karten-kopf">
|
||||
<Link href={`/hosts/${zeile.hostId}`} className="fw-semibold text-decoration-none">
|
||||
{zeile.hostname}
|
||||
</Link>
|
||||
</td>
|
||||
<td data-label="Kunde">
|
||||
{zeile.kunde ? (
|
||||
<Link href={`/kunden/${zeile.kunde.id}`} className="text-decoration-none">
|
||||
{zeile.kunde.name}
|
||||
@@ -248,13 +254,13 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
{zeile.hostname}
|
||||
</Link>
|
||||
</td>
|
||||
<td className="d-none d-lg-table-cell">
|
||||
<td className="d-none d-lg-table-cell" data-label="Laufzeit">
|
||||
<Link href={`/rechnungen/${zeile.id}`} className="text-decoration-none">
|
||||
{formatDatum(zeile.start)} – {formatDatum(zeile.ende)}
|
||||
</Link>
|
||||
{zeile.vorschlag ? <Badge farbe="info">Vorschlag</Badge> : null}
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Proxmox-Rechnung">
|
||||
<CopyableCode wert={zeile.einkaufNummer} />
|
||||
{zeile.einkaufBezahlt === false && faelligIn !== null && faelligIn <= 7 ? (
|
||||
<div className="small pl-text-danger">
|
||||
@@ -262,8 +268,10 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
</div>
|
||||
) : null}
|
||||
</td>
|
||||
<td className="d-none d-xl-table-cell">{formatEuro(zeile.einkaufBetrag)}</td>
|
||||
<td className="text-center">
|
||||
<td className="d-none d-xl-table-cell" data-label="Einkauf">
|
||||
{formatEuro(zeile.einkaufBetrag)}
|
||||
</td>
|
||||
<td className="text-center" data-label="Einkauf bezahlt">
|
||||
<StatusHaken
|
||||
laufzeitId={zeile.id}
|
||||
hostId={zeile.hostId}
|
||||
@@ -273,11 +281,13 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
titel="Proxmox-Rechnung bezahlt"
|
||||
/>
|
||||
</td>
|
||||
<td>
|
||||
<td data-label="Kundenrechnung">
|
||||
<CopyableCode wert={zeile.verkaufNummer} />
|
||||
</td>
|
||||
<td className="d-none d-xl-table-cell">{formatEuro(zeile.verkaufBetrag)}</td>
|
||||
<td className="text-center">
|
||||
<td className="d-none d-xl-table-cell" data-label="Verkauf">
|
||||
{formatEuro(zeile.verkaufBetrag)}
|
||||
</td>
|
||||
<td className="text-center" data-label="Rechnung erstellt">
|
||||
<StatusHaken
|
||||
laufzeitId={zeile.id}
|
||||
hostId={zeile.hostId}
|
||||
@@ -286,7 +296,7 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
titel="Kundenrechnung erstellt"
|
||||
/>
|
||||
</td>
|
||||
<td className="text-center">
|
||||
<td className="text-center" data-label="Vom Kunden bezahlt">
|
||||
<StatusHaken
|
||||
laufzeitId={zeile.id}
|
||||
hostId={zeile.hostId}
|
||||
@@ -297,6 +307,7 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
/>
|
||||
</td>
|
||||
<td
|
||||
data-label="Marge"
|
||||
className={`d-none d-xl-table-cell ${(zeile.marge ?? 0) >= 0 ? "pl-text-success" : "pl-text-danger"}`}
|
||||
>
|
||||
{zeile.marge === null ? "–" : formatEuro(zeile.marge)}
|
||||
@@ -306,7 +317,7 @@ export default async function RechnungenSeite({ searchParams }: { searchParams:
|
||||
})}
|
||||
{zeilen.length === 0 ? (
|
||||
<tr>
|
||||
<td colSpan={11} className="text-center text-body-secondary py-4">
|
||||
<td colSpan={12} className="text-center text-body-secondary py-4">
|
||||
Nichts gefunden. <Link href="/rechnungen?status=alle">Alle anzeigen</Link>
|
||||
</td>
|
||||
</tr>
|
||||
|
||||
@@ -382,10 +382,128 @@ body {
|
||||
transform: translateX(-50%);
|
||||
}
|
||||
|
||||
/* Zeitleiste der Jahresbahn: Breiten als Klassen, damit sie auf dem Telefon kleiner ausfallen. */
|
||||
.pl-timeline {
|
||||
min-width: 820px;
|
||||
}
|
||||
|
||||
.pl-timeline-kopf {
|
||||
margin-left: 13rem;
|
||||
}
|
||||
|
||||
.pl-timeline-name {
|
||||
width: 13rem;
|
||||
}
|
||||
|
||||
/* ---------- Mobil ---------- */
|
||||
|
||||
@media (max-width: 575.98px) {
|
||||
.pl-touch-target {
|
||||
min-height: 44px;
|
||||
}
|
||||
|
||||
/*
|
||||
* Ein Kontrollkästchen darf nicht selbst 44 px hoch werden – sonst steht in der Zeile ein
|
||||
* großer leerer Kasten. Die Fläche zum Treffen bekommt stattdessen sein Umfeld (#035).
|
||||
*/
|
||||
.pl-haken {
|
||||
min-width: 44px;
|
||||
min-height: 44px;
|
||||
}
|
||||
|
||||
.pl-haken .form-check-input {
|
||||
width: 1.4rem;
|
||||
height: 1.4rem;
|
||||
}
|
||||
}
|
||||
|
||||
/*
|
||||
* Breite Tabellen auf dem Telefon: Jede Zeile wird zur Karte, jede Zelle bekommt ihre
|
||||
* Beschriftung aus `data-label` davor. Das ersetzt das Scrollen nach rechts, bei dem die
|
||||
* Hälfte der Angaben unsichtbar bleibt (#035).
|
||||
*
|
||||
* Zellen ohne `data-label` stehen für sich – etwa eine Zeile mit Knöpfen.
|
||||
*/
|
||||
@media (max-width: 767.98px) {
|
||||
.pl-karten-tabelle,
|
||||
.pl-karten-tabelle tbody,
|
||||
.pl-karten-tabelle tr,
|
||||
.pl-karten-tabelle td {
|
||||
display: block;
|
||||
width: 100%;
|
||||
}
|
||||
|
||||
.pl-karten-tabelle thead {
|
||||
display: none;
|
||||
}
|
||||
|
||||
.pl-karten-tabelle tr {
|
||||
padding: 0.35rem 0.25rem;
|
||||
border-bottom: 1px solid var(--bs-border-color);
|
||||
}
|
||||
|
||||
.pl-karten-tabelle tr:last-child {
|
||||
border-bottom: 0;
|
||||
}
|
||||
|
||||
.pl-karten-tabelle td {
|
||||
display: flex;
|
||||
gap: 0.75rem;
|
||||
align-items: baseline;
|
||||
justify-content: space-between;
|
||||
padding: 0.3rem 0.75rem;
|
||||
text-align: right;
|
||||
border: 0;
|
||||
}
|
||||
|
||||
.pl-karten-tabelle td::before {
|
||||
flex: 0 0 auto;
|
||||
font-size: 0.8125rem;
|
||||
color: var(--bs-secondary-color);
|
||||
text-align: left;
|
||||
content: attr(data-label);
|
||||
}
|
||||
|
||||
/* Die erste Zelle trägt den Namen – sie steht als Überschrift der Karte für sich. */
|
||||
.pl-karten-tabelle td.pl-karten-kopf {
|
||||
display: block;
|
||||
padding-top: 0.6rem;
|
||||
font-size: 1.0625rem;
|
||||
text-align: left;
|
||||
}
|
||||
|
||||
.pl-karten-tabelle td.pl-karten-kopf::before {
|
||||
content: none;
|
||||
}
|
||||
|
||||
/* Lange Namen und Schlüssel sollen umbrechen, aber nicht Buchstabe für Buchstabe. */
|
||||
.pl-karten-tabelle td a,
|
||||
.pl-karten-tabelle td .pl-mono {
|
||||
overflow-wrap: anywhere;
|
||||
word-break: normal;
|
||||
}
|
||||
|
||||
/* Auf dem Telefon bleibt mehr Platz für die Monate, wenn die Namensspalte schrumpft. */
|
||||
.pl-timeline {
|
||||
min-width: 560px;
|
||||
}
|
||||
|
||||
.pl-timeline-kopf {
|
||||
margin-left: 7rem;
|
||||
}
|
||||
|
||||
.pl-timeline-name {
|
||||
width: 7rem;
|
||||
}
|
||||
|
||||
/* Kartenkopf mit Knöpfen: untereinander statt nebeneinander über den Rand hinaus. */
|
||||
.card-header {
|
||||
flex-wrap: wrap;
|
||||
gap: 0.5rem;
|
||||
}
|
||||
|
||||
.card-header .card-tools {
|
||||
flex-wrap: wrap;
|
||||
width: 100%;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -0,0 +1,86 @@
|
||||
"use client";
|
||||
|
||||
import { useState } from "react";
|
||||
import { hostKundeSetzen } from "@/server/actions/host-detail";
|
||||
|
||||
export interface KundeMitStandorten {
|
||||
id: string;
|
||||
name: string;
|
||||
standorte: { id: string; name: string }[];
|
||||
}
|
||||
|
||||
/**
|
||||
* Kunde und Standort eines Hosts ändern (#036).
|
||||
*
|
||||
* Beides gehört in **ein** Formular: Die Standorte hängen am Kunden, und ein Formular, das nur den
|
||||
* Kunden sendet, würde den Standort beim Speichern mitlöschen – genau das passierte vorher.
|
||||
*/
|
||||
export function KundeStandortFormular({
|
||||
hostId,
|
||||
kunden,
|
||||
kundeId,
|
||||
standortId,
|
||||
}: {
|
||||
hostId: string;
|
||||
kunden: KundeMitStandorten[];
|
||||
kundeId: string | null;
|
||||
standortId: string | null;
|
||||
}) {
|
||||
const [gewaehlterKunde, setGewaehlterKunde] = useState(kundeId ?? "");
|
||||
const standorte = kunden.find((kunde) => kunde.id === gewaehlterKunde)?.standorte ?? [];
|
||||
|
||||
return (
|
||||
<form action={hostKundeSetzen} className="mt-3">
|
||||
<input type="hidden" name="hostId" value={hostId} />
|
||||
|
||||
<label className="form-label small" htmlFor="kundeId">
|
||||
Kunde ändern
|
||||
</label>
|
||||
<select
|
||||
id="kundeId"
|
||||
name="kundeId"
|
||||
className="form-select form-select-sm mb-2"
|
||||
value={gewaehlterKunde}
|
||||
onChange={(e) => setGewaehlterKunde(e.target.value)}
|
||||
>
|
||||
<option value="">– nicht zugeordnet –</option>
|
||||
{kunden.map((kunde) => (
|
||||
<option key={kunde.id} value={kunde.id}>
|
||||
{kunde.name}
|
||||
</option>
|
||||
))}
|
||||
</select>
|
||||
|
||||
<label className="form-label small" htmlFor="standortId">
|
||||
Standort
|
||||
</label>
|
||||
<div className="input-group input-group-sm">
|
||||
<select
|
||||
id="standortId"
|
||||
name="standortId"
|
||||
className="form-select"
|
||||
defaultValue={standortId ?? ""}
|
||||
disabled={standorte.length === 0}
|
||||
// `key` erzwingt ein frisches Feld, wenn der Kunde wechselt – sonst bliebe die alte Auswahl stehen.
|
||||
key={gewaehlterKunde}
|
||||
>
|
||||
<option value="">– keiner –</option>
|
||||
{standorte.map((standort) => (
|
||||
<option key={standort.id} value={standort.id}>
|
||||
{standort.name}
|
||||
</option>
|
||||
))}
|
||||
</select>
|
||||
<button type="submit" className="btn btn-outline-primary">
|
||||
Setzen
|
||||
</button>
|
||||
</div>
|
||||
|
||||
{gewaehlterKunde && standorte.length === 0 ? (
|
||||
<div className="form-text">
|
||||
Für diesen Kunden gibt es noch keinen Standort. Anlegen lässt er sich auf der Kundenseite.
|
||||
</div>
|
||||
) : null}
|
||||
</form>
|
||||
);
|
||||
}
|
||||
@@ -37,7 +37,7 @@ export function StatusHaken({
|
||||
}
|
||||
|
||||
return (
|
||||
<span className="d-inline-flex align-items-center justify-content-center">
|
||||
<span className="pl-haken d-inline-flex align-items-center justify-content-center">
|
||||
{laeuft ? (
|
||||
<span
|
||||
className="spinner-border spinner-border-sm text-body-secondary"
|
||||
@@ -47,7 +47,7 @@ export function StatusHaken({
|
||||
) : (
|
||||
<input
|
||||
type="checkbox"
|
||||
className="form-check-input pl-touch-target"
|
||||
className="form-check-input"
|
||||
checked={wert}
|
||||
disabled={deaktiviert}
|
||||
onChange={(e) => umschalten(e.target.checked)}
|
||||
|
||||
@@ -57,8 +57,24 @@ export async function hostKundeSetzen(formular: FormData) {
|
||||
const standortId = formular.get("standortId")?.toString() || null;
|
||||
if (!hostId) return;
|
||||
|
||||
await prisma.host.update({ where: { id: hostId }, data: { kundeId, standortId } });
|
||||
await protokolliere("host.zugewiesen", "Host", hostId, { kundeId, standortId });
|
||||
// Ein Standort gehört immer zu einem Kunden. Passt beides nicht zusammen – etwa weil der Kunde
|
||||
// gewechselt hat –, wird der Standort verworfen statt eine falsche Zuordnung zu speichern.
|
||||
const gueltigerStandort =
|
||||
standortId && kundeId
|
||||
? ((await prisma.standort.findFirst({
|
||||
where: { id: standortId, kundeId },
|
||||
select: { id: true },
|
||||
})) ?? null)
|
||||
: null;
|
||||
|
||||
await prisma.host.update({
|
||||
where: { id: hostId },
|
||||
data: { kundeId, standortId: gueltigerStandort?.id ?? null },
|
||||
});
|
||||
await protokolliere("host.zugewiesen", "Host", hostId, {
|
||||
kundeId,
|
||||
standortId: gueltigerStandort?.id ?? null,
|
||||
});
|
||||
|
||||
if (kundeId) {
|
||||
await prisma.alert.updateMany({
|
||||
|
||||
@@ -0,0 +1,94 @@
|
||||
import { afterAll, beforeEach, describe, expect, it, vi } from "vitest";
|
||||
import { datenbankLeeren, datenbankVerfuegbar, testPrisma } from "./helpers/db";
|
||||
|
||||
/** Zuweisung von Kunde und Standort an einen Host (#036). */
|
||||
vi.mock("@/server/auth/sitzungspflicht", () => ({
|
||||
sitzungErforderlich: async () => ({ benutzerId: "test", rolle: "ADMIN", kundeId: null }),
|
||||
sichtbarerKunde: async () => null,
|
||||
darfEinkaufSehen: async () => true,
|
||||
angemeldet: async () => null,
|
||||
}));
|
||||
vi.mock("next/cache", () => ({ revalidatePath: () => {} }));
|
||||
|
||||
const { hostKundeSetzen } = await import("@/server/actions/host-detail");
|
||||
|
||||
async function aufbau() {
|
||||
const [kundeA, kundeB] = await Promise.all([
|
||||
testPrisma.kunde.create({ data: { name: "Kunde A", kundennummer: "K-A" } }),
|
||||
testPrisma.kunde.create({ data: { name: "Kunde B", kundennummer: "K-B" } }),
|
||||
]);
|
||||
const [standortA, standortB] = await Promise.all([
|
||||
testPrisma.standort.create({ data: { kundeId: kundeA.id, name: "Rechenzentrum A" } }),
|
||||
testPrisma.standort.create({ data: { kundeId: kundeB.id, name: "Rechenzentrum B" } }),
|
||||
]);
|
||||
const host = await testPrisma.host.create({
|
||||
data: { agentId: "a-1", hostname: "pve-01", produkt: "PVE" },
|
||||
});
|
||||
return { kundeA, kundeB, standortA, standortB, host };
|
||||
}
|
||||
|
||||
function formular(werte: Record<string, string>) {
|
||||
const daten = new FormData();
|
||||
for (const [name, wert] of Object.entries(werte)) daten.set(name, wert);
|
||||
return daten;
|
||||
}
|
||||
|
||||
describe.runIf(datenbankVerfuegbar)("Kunde und Standort setzen", () => {
|
||||
beforeEach(async () => {
|
||||
await datenbankLeeren();
|
||||
});
|
||||
|
||||
afterAll(async () => {
|
||||
await testPrisma.$disconnect();
|
||||
});
|
||||
|
||||
it("setzt Kunde und Standort zusammen", async () => {
|
||||
const { kundeA, standortA, host } = await aufbau();
|
||||
|
||||
await hostKundeSetzen(formular({ hostId: host.id, kundeId: kundeA.id, standortId: standortA.id }));
|
||||
|
||||
const danach = await testPrisma.host.findUniqueOrThrow({ where: { id: host.id } });
|
||||
expect(danach.kundeId).toBe(kundeA.id);
|
||||
expect(danach.standortId).toBe(standortA.id);
|
||||
});
|
||||
|
||||
it("verwirft einen Standort, der zu einem anderen Kunden gehört", async () => {
|
||||
const { kundeA, standortB, host } = await aufbau();
|
||||
|
||||
await hostKundeSetzen(formular({ hostId: host.id, kundeId: kundeA.id, standortId: standortB.id }));
|
||||
|
||||
const danach = await testPrisma.host.findUniqueOrThrow({ where: { id: host.id } });
|
||||
expect(danach.kundeId).toBe(kundeA.id);
|
||||
expect(danach.standortId).toBeNull();
|
||||
});
|
||||
|
||||
it("löst die Zuordnung samt Standort, wenn kein Kunde gewählt ist", async () => {
|
||||
const { kundeA, standortA, host } = await aufbau();
|
||||
await hostKundeSetzen(formular({ hostId: host.id, kundeId: kundeA.id, standortId: standortA.id }));
|
||||
|
||||
await hostKundeSetzen(formular({ hostId: host.id, kundeId: "", standortId: "" }));
|
||||
|
||||
const danach = await testPrisma.host.findUniqueOrThrow({ where: { id: host.id } });
|
||||
expect(danach.kundeId).toBeNull();
|
||||
expect(danach.standortId).toBeNull();
|
||||
});
|
||||
|
||||
it("schließt die Warnung „Host ohne Kunde“ beim Zuweisen", async () => {
|
||||
const { kundeA, host } = await aufbau();
|
||||
await testPrisma.alert.create({
|
||||
data: {
|
||||
regel: "host-ohne-kunde",
|
||||
fingerprint: `host-ohne-kunde:${host.id}`,
|
||||
schwere: "INFO",
|
||||
titel: "Host ohne Kunde",
|
||||
beschreibung: "…",
|
||||
hostId: host.id,
|
||||
},
|
||||
});
|
||||
|
||||
await hostKundeSetzen(formular({ hostId: host.id, kundeId: kundeA.id }));
|
||||
|
||||
const alert = await testPrisma.alert.findFirstOrThrow({ where: { hostId: host.id } });
|
||||
expect(alert.erledigtAm).not.toBeNull();
|
||||
});
|
||||
});
|
||||