Skip to content

Database models 30 controller#45

Draft
MareWalzl wants to merge 21 commits into
database_models-30from
database_models-30_controller
Draft

Database models 30 controller#45
MareWalzl wants to merge 21 commits into
database_models-30from
database_models-30_controller

Conversation

@MareWalzl

Copy link
Copy Markdown
Contributor

No description provided.

MareWalzl and others added 8 commits April 19, 2023 14:43
* WIP: update openapi spec to fit the business model

* day api

* add timelots, room and session to openapi

* add new paths to openapi

* split openapi.yml into multiple sub files

---------

Co-authored-by: Lukas Bauer <lukas.bauer@mayflower.de>
Co-authored-by: Laura Schlemmer <laura.schlemmer@mayflower.de>
@MareWalzl MareWalzl self-assigned this Apr 21, 2023
@kaidobit

kaidobit commented Apr 24, 2023

Copy link
Copy Markdown

The Delegate-Pattern haven't been implemented correctly:

The openapi-plugin generates the Controller-classes already for us, these have a class (the Service-class written by us) injected, which implements the Delegate-interface.

Therefore you should call these classes e.g. RoomService.

@kaidobit kaidobit closed this Apr 24, 2023
@kaidobit kaidobit reopened this Apr 24, 2023
@kaidobit

Copy link
Copy Markdown

Annotating the Service-classes with @Component is fine, however @Service is better. this is a purely cosmetic change, there is technically no difference.

@MareWalzl MareWalzl force-pushed the database_models-30_controller branch from e678b4a to bf93502 Compare April 25, 2023 09:32
@MareWalzl MareWalzl force-pushed the database_models-30_controller branch from bf93502 to 8488508 Compare April 25, 2023 09:42
@kaidobit kaidobit force-pushed the database_models-30_controller branch from d4a274e to 396ffee Compare April 25, 2023 13:35
@kaidobit kaidobit force-pushed the database_models-30_controller branch from 593d9a7 to b483d87 Compare April 28, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants