-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixup: Remove DTO PR #348
Fixup: Remove DTO PR #348
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delivery und Mandate müssen optional sein, weil sonst die Requests aus dem Admin-UI fehlschlagen. Da wir leider immer noch keine e2e-Tests für die Adminseite haben, fällt dies hier im PR nicht auf :(
Der StrictMode
musste damals entfernt werden, weil da noch die User mit geseeded wurden. Da die inzwischen rausgefallen sind, kann der so drin bleiben..
Ich konnte im Admin-UI keinen Fehler produzieren. Wie sind hier die Repro steps?
Die User werden auch heute noch geseeded, jedoch gibt es das berechnete Property |
Fixes Microsoft.EntityFrameworkCore.Model.Validation[10622]
#232 hat:
Aufgefallen bei der Umsetzung von #343