Repository navigation
fix: correct existing time entries through versioned requests (#43) - #48
patrickschiller wants to merge 6 commits into
Conversation
Reopen returned drafts with their values and revision reason. Add owner-only versioned editing/resubmission, shared validation, balance/overlap self-exclusion, transactional concurrency guards, employee names, synchronized API contracts and regression/manual acceptance coverage. Co-authored-by: patrickschiller <39913537+patrickschiller@users.noreply.github.com> Signed-off-by: Patrick Schiller <39913537+patrickschiller@users.noreply.github.com>
Keep persisted timestamps unchanged until their time field is edited. Resolve wall-clock validation and payloads to identical instants, require explicit ambiguous-time choices, reject clock-change gaps, and cover seconds, both fold occurrences and reason-only save/resubmit cycles. Co-authored-by: patrickschiller <39913537+patrickschiller@users.noreply.github.com> Signed-off-by: Patrick Schiller <39913537+patrickschiller@users.noreply.github.com>
Keep the English DST selection regression independent of jsdom persisted locale on Node 20. Verify all 208 web tests with Node 20.20.2 and document the CI follow-up. Co-authored-by: patrickschiller <39913537+patrickschiller@users.noreply.github.com> Signed-off-by: Patrick Schiller <39913537+patrickschiller@users.noreply.github.com>
Add scoped, versioned TimeEntry review with distinct current-manager and HR stages, durable decision audit, legacy pending support and conservative edit invalidation. Surface employee guidance, reviewer actions and history in German/English, synchronize API contracts, and verify additive database upgrades and security/concurrency regressions. Based on accepted #42; no release or merge. Co-authored-by: patrickschiller <39913537+patrickschiller@users.noreply.github.com> Signed-off-by: Patrick Schiller <39913537+patrickschiller@users.noreply.github.com>
Bind correction requests to an owned eligible entry ID and original revision with immutable server snapshots. Revalidate targets, overlaps and required manager/HR stages, then update the same entry and linked audit atomically at final approval. Retain draft precision, metadata and break policies; add target selection/diffs, upgrade safety and focused rollback/concurrency regressions. Cumulative on accepted #42 and #41; no release or merge. Co-authored-by: patrickschiller <39913537+patrickschiller@users.noreply.github.com> Signed-off-by: Patrick Schiller <39913537+patrickschiller@users.noreply.github.com>
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 34a9b50945
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| private dto(row: ReviewEntry, actor: Employee): TimeEntryApprovalDto { | ||
| const state = frameApprovalState(row); | ||
| return { | ||
| entry: toTimeEntryDto(row), |
There was a problem hiding this comment.
For split bookings with a captureGroupId, this calls toTimeEntryDto without overriding its per-entry summary, so the new approval screen displays incorrect break deductions and net time. For example, two four-hour segments under the existing six-hour/30-minute break policy each display zero break instead of their allocated 15 minutes. TimeEntryApprovalRow renders these values to both employees and reviewers, contradicting the booking and account totals. Load all capture-group members, including those outside the approval-list filter, and use calculateCaptureSummaries when building these DTOs.
Useful? React with 👍 / 👎.
Distinguish forbidden foreign targets from ignored body employee identities; eligible own targets remain valid under the authenticated identity. Co-authored-by: patrickschiller <39913537+patrickschiller@users.noreply.github.com> Signed-off-by: Patrick Schiller <39913537+patrickschiller@users.noreply.github.com>
Fixes #43.
Abhängigkeit und Reihenfolge
Depends on #46 und #47. Basis ist der unabhängig akzeptierte #41-Head
dbe68f57a93ba26bdc315d31fd4b313586a4337b. Der PR zielt bewusst aufmain,damit sämtliche bestehenden CI-Jobs laufen; gegen main ist der Diff
kumulativ #42 + #41 + #43.
Vorgesehene Reihenfolge: #46 → #47 → dieser PR. Kein Merge und kein
Auto-Merge. Der obere Branch kann alle drei Pakete bereits gemeinsam testen.
Nur #43: Commit
34a9b5094568fffd990fa0827e6e9f8c28c13209,24 Dateien, +3402/−83. Kumulativ gegen main: 37 Dateien, +8959/−407.
Umsetzung #43
Berechtigte eigene, abgeschlossene, nicht verworfene Team-Buchungen sind direkt
auswählbar; Zielwerte werden einschließlich Sekunden/DST-Vorkommen vorbelegt.
Beschäftigte und Genehmigende sehen Vorher/Vorschlag, lokale Zeit mit Offset,
Bruttodifferenz, Ziel-ID und gebundene Originalrevision.
den unveränderlichen Originalsnapshot. Fremde/offene/verworfene/abgelehnte/
Solo-Ziele und unvollständige/gefälschte Bindungen werden abgewiesen.
Identität stammt aus dem JWT, nicht aus Body-/Query-Mitarbeiterfeldern.
Zielbindung und Originalversion; kein stilles Retargeting oder Auffrischen
unter einer alten Freigabe. Stale-Versionen verlangen Prüfung und einen neuen
Antrag auf dem aktuellen Buchungsstand.
ausgenommen. Alle anderen Buchungen bleiben geprüft; kein Tagesüberschreiben.
Bereits materialisierte genehmigte Zeitkorrekturanträge reservieren nicht
zusätzlich ihren alten Zeitraum — die tatsächlichen Buchungen sind maßgeblich.
Request-Entscheidung und Audit in einer Transaktion. Keine neue Doppelbuchung.
Gemeinsame Sperrreihenfolge, Pflichtrevision und atomarer Versionsvergleich
schützen gegen Clock-/Split-/Metadaten-/[bug] Genehmigung von Buchungen außerhalb der Rahmenzeit #41-Freigaben, konkurrierende Anträge,
Wiederholung und Rollback nach bereits ausgeführtem Update.
Selbst-/Fremdfreigabe ist ausgeschlossen. Vorgeschlagene Außenrahmenwerte
erzwingen zusätzlich HR; alte Buchungsfreigaben autorisieren keine neuen
Zeiten. Erst die letzte erforderliche Stufe wendet die Änderung an.
Pausenpolitik bleiben erhalten; Summaries werden neu berechnet. Ursprüngliche
Auditdaten bleiben bestehen; neues Vorher/Nachher-Audit enthält tatsächlichen
Akteur, Zeit und Request-FK. Antragsberechtigte sehen es über einen eigenen
Read-Endpunkt; [bug] Genehmigung von Buchungen außerhalb der Rahmenzeit #41-Verlauf zeigt den verknüpften Korrekturantrag.
Request-Storno irreführend als rückgängig dargestellt werden.
bleiben getrennt erhalten. DE/EN, benutzergebundene Query-Caches,
Realtime-Invalidierung, synchroner OpenAPI-/Client-Vertrag.
20261107040000_targeted_time_corrections; keineAlt-Migration geändert, keine erfundenen Ziel-/Auditlinks für Bestandsdaten.
Lokale Verifikation
Isolierter PostgreSQL-16-Container mit ausschließlich synthetischen Datenbanken;
keine vorhandene Datenbank zurückgesetzt. pnpm 9.15.9, vollständige Testläufe
zusätzlich unter CI-Node 20.20.2.
TZ=Europe/Berlin pnpm nx e2e api-e2eunter Node 20.20.2TZ=UTC pnpm nx run-many -t test -p web,shared --run --skip-nx-cacheunter Node 20.20.2pnpm nx run-many -t typecheck,lint -p api,web,shared,api-e2epnpm nx run-many -t build -p api,web,shared, getrennt von Typecheckspnpm verify:api/git diff --checkBelegt sind 08–16→08–17 und 09–16 mit gleicher ID/stabiler Anzahl, echte
Pausen-/Saldodifferenzen, Gruppensnapshots, Rechte/Modus, unveränderliche
reason-only Drafts, Sekunden/DST, konkurrierende unterschiedliche Anträge und
tatsächlicher Rollback nach bereits erfolgtem Entry-Update für alle finalen Routen.
Nachtest und Grenzen
Deutsche Anleitung, Paket #43: isoliertes Setup, synthetische Rollen,
reproduzierbare Originalbuchungen, Positiv-/Negativ-/Konkurrenzfälle, Erwartungen
und tatsächliche Ergebnisse.
Die UI-Regressionen sind React-Testing-Library-Tests; der dokumentierte manuelle
Docker-/Browser-Durchlauf wurde nicht als durchgeführt behauptet.
Nur #42/#41/#43. #45 und #44 nicht begonnen. Kein Release, Tag, Versionsbump,
Merge, Deployment oder Release-Workflow.
GitHub-Gates auf
34a9b5094568fffd990fa0827e6e9f8c28c13209vollständig grün:build-test-lint,api-e2e,database-upgradeundterminal-pin-browsererfolgreich.pnpm verify:apierfolgreich.Builds erfolgreich.
erfolgreich; verlangter Signed-off-by- und exakter Co-Author-Trailer vorhanden.