Skip to content
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

devops: make it so type gen doesn't have every single field marked as optional lol #105

Open
LinusBolls opened this issue Nov 21, 2024 · 5 comments
Assignees

Comments

@LinusBolls
Copy link
Collaborator

No description provided.

@Laurin-Notemann
Copy link
Owner

@Thiies btw wie man in #110 sehen kann müssen bei jedem NotNull type der decerator in den DTOs geadded werden, weil der bei dem openapi schena gen sonst das field required nicht added, aber ich schau nochmal etwas weiter ob es da andere WEge gibt aber erstmal sind wir stuck damit

@Thiies
Copy link
Collaborator

Thiies commented Nov 21, 2024

wir können @NotNull auch einfach rausnehmen. Das ist eh größtenteils deko

@Thiies
Copy link
Collaborator

Thiies commented Nov 21, 2024

und wir null checken eh fast alles

@Laurin-Notemann
Copy link
Owner

Nee das ist nicht das Problem, das ding ist, dass der Code der für linus genieriert wird sieht halt so aus als wäre alles optional also nullable, ich hatte das @NotNull da zum test hingeschrieben ob das wirklich so klappen würde, macht einfach das typing etwas nerviger im Frontend

@Thiies
Copy link
Collaborator

Thiies commented Nov 22, 2024

mach nen issue wenn du ne lösung gefunden hast

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

No branches or pull requests

3 participants