Export ohne Vier-Augen, Wiedervorlage setzbar, Druckansicht mit Weg zurück
**Export.** Das Vier-Augen-Verfahren ab Stufe 3 ist weg. Wer exportiert, entscheidet jetzt allein die Freigabestufe: mindestens die des Ziels. Das ist strenger als das Lesen - wer als Beteiligter ausnahmsweise in eine höher eingestufte Akte darf, darf sie deshalb noch nicht ausser Haus geben. Dabei fiel eine Lücke auf: hängte man sich an einen laufenden Auftrag, entstand kein Protokolleintrag. „Jeder Export wird protokolliert" heisst jeder Abruf, nicht jeder Rendervorgang - wer dieselbe Datei bekommt, muss genauso darin stehen. **Fristen.** dueAt wurde überall gelesen - Übersicht, Suchfilter, Ausdruck - und nirgends gesetzt. Es gab keine Stelle, an der man ein Datum einträgt, deshalb stand unter „Fristen" immer „Nichts steht an". Jetzt im Kopf der Akte, mit Protokolleintrag und ohne Verlaufseintrag: eine Wiedervorlage ist eine Merkhilfe, keine Aussage über den Fall. **Statuswechsel.** Aus einem Entwurf führt laut Lebenszyklus nur „Offen" heraus, deshalb standen dort nie mehr als zwei Ziele. Das sah aus wie ein Fehler und war keiner - jetzt steht die Erklärung daneben und an jeder Option. **Druckansicht.** Sie hatte kein eigenes Layout: das weisse Blatt lag auf dem dunklen Thema und es gab keinen Weg zurück. Jetzt heller Grund, eine Leiste mit Zurück und Drucken, beides beim Drucken ausgeblendet. Das Ziel wird aus dem Pfad abgeleitet, damit es auch beim direkten Aufruf stimmt. **Abteilungsstatistik.** Eine Abteilung ohne Freigabe wird als „Ohne Freigabe" geführt statt weggelassen - sonst summierten sich die Anteile nicht auf die Zahl daneben. Mehrere ergeben ein Stück, sonst stünde dort, wie viele es sind. Dazu: Dienstgrad in eckigen Klammern vor dem Namen und in dessen Grösse, in einer eigenen Spalte; Knopfform für „Personalakte anlegen"; die abgeschnittene Anzahl im Kopf der Personalaktenliste; „Zuletzt geöffnet" ohne die zwei Hinweistexte, leer als Zeichen statt als Satz.
This commit is contained in:
@@ -1,117 +0,0 @@
|
||||
'use client'
|
||||
|
||||
import { useState, useTransition } from 'react'
|
||||
import { exportText } from '@/lib/i18n/export'
|
||||
import { commonText } from '@/lib/i18n/common'
|
||||
import { approveExportAction, rejectExportAction } from './actions'
|
||||
|
||||
type Auftrag = {
|
||||
id: string
|
||||
kind: string
|
||||
label: string
|
||||
requestedBy: string
|
||||
clearanceAtTime: number
|
||||
createdAt: string
|
||||
}
|
||||
|
||||
/**
|
||||
* Offene Exportfreigaben.
|
||||
*
|
||||
* Vier-Augen-Prinzip: hier steht nur, was **jemand anderes** angefragt hat.
|
||||
* Die eigene Anfrage taucht gar nicht erst auf, und der Dienst weist sie auch
|
||||
* dann zurück, wenn jemand die Kennung von Hand einsetzt.
|
||||
*/
|
||||
export function Approvals({ jobs }: { jobs: Auftrag[] }) {
|
||||
const [ablehnung, setAblehnung] = useState<string | null>(null)
|
||||
const [grund, setGrund] = useState('')
|
||||
const [fehler, setFehler] = useState<string | null>(null)
|
||||
const [pending, startTransition] = useTransition()
|
||||
|
||||
if (jobs.length === 0) {
|
||||
return <p className="text-text-dim">{exportText.approvalsEmpty}</p>
|
||||
}
|
||||
|
||||
return (
|
||||
<div>
|
||||
<ul className="border-t border-line">
|
||||
{jobs.map((auftrag) => (
|
||||
<li key={auftrag.id} className="border-b border-line py-2">
|
||||
<div className="record-row text-2xs">
|
||||
<span className="record-label">
|
||||
{exportText.kinds[auftrag.kind] ?? auftrag.kind}
|
||||
</span>
|
||||
<span className="record-key">{auftrag.label}</span>
|
||||
<span className="record-title text-text-dim">
|
||||
{exportText.requestedBy} {auftrag.requestedBy}
|
||||
</span>
|
||||
<button
|
||||
type="button"
|
||||
disabled={pending}
|
||||
onClick={() =>
|
||||
startTransition(async () => {
|
||||
const ergebnis = await approveExportAction({ id: auftrag.id })
|
||||
if (ergebnis.error) setFehler(ergebnis.error)
|
||||
})
|
||||
}
|
||||
className="btn btn-sm btn-primary"
|
||||
>
|
||||
{exportText.approve}
|
||||
</button>
|
||||
<button
|
||||
type="button"
|
||||
onClick={() => setAblehnung(auftrag.id)}
|
||||
className="btn btn-sm"
|
||||
>
|
||||
{exportText.reject}
|
||||
</button>
|
||||
</div>
|
||||
|
||||
{ablehnung === auftrag.id ? (
|
||||
<div className="mt-2">
|
||||
<textarea
|
||||
value={grund}
|
||||
onChange={(event) => setGrund(event.target.value)}
|
||||
rows={2}
|
||||
placeholder={exportText.rejectReason}
|
||||
aria-label={exportText.rejectReason}
|
||||
className="block w-full max-w-sm field text-2xs"
|
||||
/>
|
||||
<div className="mt-1 flex gap-3 text-2xs">
|
||||
<button
|
||||
type="button"
|
||||
disabled={pending || grund.trim().length < 10}
|
||||
onClick={() =>
|
||||
startTransition(async () => {
|
||||
const ergebnis = await rejectExportAction({
|
||||
id: auftrag.id,
|
||||
reason: grund,
|
||||
})
|
||||
if (ergebnis.error) setFehler(ergebnis.error)
|
||||
else {
|
||||
setAblehnung(null)
|
||||
setGrund('')
|
||||
}
|
||||
})
|
||||
}
|
||||
className="btn btn-sm text-stamp-4"
|
||||
>
|
||||
{exportText.reject}
|
||||
</button>
|
||||
<button
|
||||
type="button"
|
||||
onClick={() => setAblehnung(null)}
|
||||
className="btn btn-sm"
|
||||
>
|
||||
{commonText.cancel}
|
||||
</button>
|
||||
</div>
|
||||
</div>
|
||||
) : null}
|
||||
</li>
|
||||
))}
|
||||
</ul>
|
||||
|
||||
{fehler ? <p className="mt-2 font-condensed text-2xs text-stamp-4">{fehler}</p> : null}
|
||||
</div>
|
||||
)
|
||||
}
|
||||
@@ -1,12 +1,9 @@
|
||||
'use server'
|
||||
|
||||
import { revalidatePath } from 'next/cache'
|
||||
import { z } from 'zod'
|
||||
import { requireUser } from '@/lib/auth/current'
|
||||
import { AccessError } from '@/lib/errors'
|
||||
import {
|
||||
approveExport,
|
||||
rejectExport,
|
||||
requestExport,
|
||||
runExport,
|
||||
type ExportTarget,
|
||||
@@ -52,36 +49,4 @@ export async function requestExportAction(
|
||||
}
|
||||
}
|
||||
|
||||
const idSchema = z.object({ id: z.string().min(1) })
|
||||
|
||||
export async function approveExportAction(input: unknown): Promise<ExportState> {
|
||||
const user = await requireUser()
|
||||
const parsed = idSchema.safeParse(input)
|
||||
if (!parsed.success) return { error: de.actionErrors.default! }
|
||||
|
||||
try {
|
||||
await approveExport(user.permissions, parsed.data.id)
|
||||
// Erst nach der Freigabe wird gerendert.
|
||||
void runExport(parsed.data.id)
|
||||
revalidatePath('/uebersicht')
|
||||
return { jobId: parsed.data.id, status: 'QUEUED' }
|
||||
} catch (error) {
|
||||
return { error: toMessage(error) }
|
||||
}
|
||||
}
|
||||
|
||||
const rejectSchema = idSchema.extend({ reason: z.string().min(10).max(500) })
|
||||
|
||||
export async function rejectExportAction(input: unknown): Promise<ExportState> {
|
||||
const user = await requireUser()
|
||||
const parsed = rejectSchema.safeParse(input)
|
||||
if (!parsed.success) return { error: de.actionErrors['reason-required']! }
|
||||
|
||||
try {
|
||||
await rejectExport(user.permissions, parsed.data.id, parsed.data.reason)
|
||||
revalidatePath('/uebersicht')
|
||||
return { jobId: parsed.data.id, status: 'REJECTED' }
|
||||
} catch (error) {
|
||||
return { error: toMessage(error) }
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user