Skip to content

[bug] Bearbeiten von Anträgen #42

New issue
New issue
Open
#46
Open
[bug] Bearbeiten von Anträgen#42
#46
Labels
bugSomething isn't working

Description

@pittiplatschpitti-a11y
pittiplatschpitti-a11y
opened on Nov 6, 2026

Describe the bug

Wird ein Mitarbeiterantrag (z. B. Urlaubsantrag) durch den Vorgesetzten zur Änderung zurückdelegiert, erscheint er beim Mitarbeiter zwar in der Übersicht mit dem Status/Button „Entwurf“, dieser lässt sich jedoch überhaupt nicht mehr öffnen. Ein Klick auf den Entwurf bzw. den Button löst keinerlei Aktion aus, wodurch der Mitarbeiter den Antrag weder einsehen noch bearbeiten kann.
Reproduction steps

  1. Als Mitarbeiter einen Antrag (z. B. Urlaubsantrag) erstellen und zur Genehmigung einreichen.
  2. Als Vorgesetzter den Antrag aufrufen, zurückdelegieren und um Änderung bitten.
  3. Als Mitarbeiter die Übersicht der Anträge aufrufen: Der Antrag wird dort mit dem Status/Button „Entwurf“ aufgeführt.
  4. Auf den Entwurf-Button bzw. den Antragseintrag klicken, um diesen zu öffnen und die gewünschten Änderungen vorzunehmen. Dabei passiert dann aber nichts.

Expected behaviour

Ein Klick auf den Entwurf öffnet die Antragsmaske im Bearbeitungsmodus, sodass der Mitarbeiter die angeforderten Änderungen vornehmen und den Antrag erneut einreichen kann.
Außerdem wäre es schön, wenn der Name des Mitarbeiters irgendwo im Antrag auftaucht (und nicht separat dazu geschrieben werden muss).

Activity

  1. pittiplatschpitti-a11y
    added
    bugSomething isn't working
    on Nov 6, 2026

  2. patrickschiller commented on Nov 7, 2026

    @patrickschiller
    patrickschiller
    on Nov 7, 2026
    Owner

    Implementiert in PR #46: #46

    Belegt lokal geprüft (Commit 230dbb0e0ce7267d63cd440df900c56193736094):

    • 75 API-E2E-Tests in 8 Suites bestanden, darunter 20 neue Revisionsfälle: Manager-Rückgabe und Vertretungsablehnung → Bearbeiten → Wiedereinreichen, identische ID/erhaltene Historie und Anlagen, Namen, Fremdzugriff/Nicht-Draft, Saldo/Überlappung sowie parallele/veraltete Schreibzugriffe und alte Managerentscheidungen.
    • 190 Web-Tests und 88 Shared-Domain-Tests bestanden.
    • API/Web/Shared-Builds, alle vier Typechecks, OpenAPI-/Client-Synchronisationsprüfung und Diff-Prüfung erfolgreich. Lint ohne Fehler; ausschließlich 13 bestehende Warnungen in unveränderten Dateien.
    • Der separate Web-Typecheck behebt den im kombinierten parallelen lokalen Build-/Typecheck-Lauf beobachteten dist-Output-Konflikt; keine Projektkonfiguration dafür verändert.

    Deutsche Nachtestanleitung mit isoliertem lokalen Docker-Setup, synthetischen Rollen und Negativtests. Der manuelle Docker-/Browser-Durchlauf ist als Anleitung bereitgestellt, nicht als durchgeführt behauptet.

    Keine Veröffentlichung, kein Tag, Versionsbump, Merge oder Deployment. Nur Paket #42; Folgepakete bleiben getrennt.

    GitHub-Prüfungen ebenfalls vollständig grün: CI-Lauf mit build-test-lint, vollständiger api-e2e-Suite, database-upgrade und terminal-pin-browser; zusätzlich DCO. Alle fünf PR-Checks erfolgreich.

  3. patrickschiller
    added 2 commits that reference this issue on Nov 7, 2026

    fix: preserve correction instants across DST edits (#42)

    80c8622

    test: isolate correction editor language preference (#42)

    4ff247b

  4. patrickschiller commented on Nov 7, 2026

    @patrickschiller
    patrickschiller
    on Nov 7, 2026
    Owner

    #42: P2-Nachkorrektur der Zeitkorrektur-Bearbeitung

    PR: #46
    Finaler Head: 4ff247bc18adfc0b245d8dbb044f7968cf2a4678

    Der bei der unabhängigen Abnahme gefundene Sekunden-/DST-Roundtripfehler ist korrigiert:

    • Unveränderte Zeitfelder behalten exakt ihre gespeicherten ISO-Zeitpunkte, einschließlich Sekunden, Millisekunden und DST-Vorkommen. Eine reine Begründungsänderung verschiebt keine Zeit.
    • Bereichsprüfung und Payload nutzen dieselben Zeitpunkte; das gültige Herbst-Intervall 02:50 UTC+02:00 → 02:10 UTC+01:00 wird akzeptiert.
    • Bei geänderten doppelt vorkommenden Ortszeiten ist eine zugängliche DE/EN-Auswahl mit UTC-Offset erforderlich. Leere Zeitfelder und nicht existierende Frühjahrslücken werden abgewiesen.

    Nachweise: 18 neue Regressionen; vier Roundtriptests gegen den alten Editor erwartungsgemäß rot, mit Fix grün. Gesamte Web-Suite 208/208 unter Node 26 und zusätzlich Node 20.20.2 ohne Nx-Cache. Web-Typecheck, Web-/PWA-Build und Lint erfolgreich (0 Fehler, nur 2 bestehende Warnungen); git diff --check erfolgreich.

    Der erste CI-Nachlauf zeigte eine nur im neuen Englisch-Test verbliebene deutsche Storage-Präferenz (207/208). Die Testfixture ist jetzt isoliert; keine Produktänderung dafür nötig.

    Finale CI auf 4ff247bc18adfc0b245d8dbb044f7968cf2a4678 vollständig grün:

    • CI: build-test-lint, api-e2e, database-upgrade, terminal-pin-browser erfolgreich.
    • Vollständige API-E2E: 304/304 Tests, 37 Suites; pnpm verify:api erfolgreich.
    • CI-Web: 208/208 Tests, 34 Dateien; vier Projekt-Typechecks und relevante Builds erfolgreich.
    • DCO erfolgreich; beide Korrekturcommits mit DCO und verlangtem Co-Author-Trailer.

    Deutsche Nachtestanleitung, Abschnitt D und belegte Ergebnisse. Manueller Docker-/Browser-Nachtest nicht als durchgeführt behauptet. Keine API-/OpenAPI-/Datenbankänderung in dieser Nachkorrektur.

    Nur #42: kein Release, Tag, Versionsbump, Merge oder Deployment; #41 noch nicht begonnen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions