search

DevNico / WS-2022

2 stars 0 forks source link

feat(backend): add endpoints for OrganisationUser and OrganisationRole #42

Closed cerphilipp closed 2 years ago

cerphilipp commented 2 years ago

Hoffe das passt so

MarkusJx commented 2 years ago

Ansonsten fällt mir nichts auf, aber ich würde gerne auf die Meinung von @DevNico warten. Zusätzlich müssten wir wahrscheinlich die OrganizationUser ins Keycloak mappen, das wäre aber eine Aufgabe für einen späteren Zeitpunkt, schätze ich. @DevNico Kriegen wir die User-Ids über die request vom Keycloak wieder?

cerphilipp commented 2 years ago

Habe jetzt haufenweise Änderungen vorgenommen, hab auch die Routes in eine extra statische Klasse gezogen.

cerphilipp commented 2 years ago

@MarkusJx @DevNico Bitte um Meinung. Werde aber heute nicht mehr erreichbar sein erst morgen

MarkusJx commented 2 years ago

Allerdings mal ne generelle Frage: Wollen wir die ApiResponses mit swagger dokumentieren?

DevNico commented 2 years ago

Allerdings mal ne generelle Frage: Wollen wir die ApiResponses mit swagger dokumentieren?

Ja, aber nein aber wie. Also sinnvoll isses bestimmt. Mehrwert ~ Nutzen ist schwierig

MarkusJx commented 2 years ago

Es muss definitiv nochmal jemand durchgehen und die responses dokumentieren, wir verschieben das mal nach hinten: #58