Skip to content

Movie component#21

Merged
magail merged 4 commits intosolution-0-setupfrom
movie_component
Sep 24, 2021
Merged

Movie component#21
magail merged 4 commits intosolution-0-setupfrom
movie_component

Conversation

@magail
Copy link
Copy Markdown
Collaborator

@magail magail commented Sep 16, 2021

No description provided.

@magail magail closed this Sep 16, 2021
@magail magail reopened this Sep 16, 2021
CREATE TABLE MOVIE_GENRE (
MOVIE_ID BIGINT NOT NULL,
GENRE VARCHAR2(50 CHAR) NOT NULL,
CONSTRAINT UC_MOVIE_GENRE UNIQUE (MOVIE_ID, GENRE)
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Brauchen wir hier noch einen Constraint auf die Werte von Genre?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finde ich nicht, das passt aus meiner Sicht.

@hohwille
Copy link
Copy Markdown
Collaborator

hohwille commented Sep 23, 2021

Sobald das gemerged ist, soll ein weiteres Projekt vorkonfiguriert sein und zwar hier:
https://github.com/devonfw-training/java-foundations-settings/tree/master/projects

Fragen/Vorschläge:

  1. Soll dieser PR nicht gleich auf master und von dort auf alle solution Branches?
  2. Ich würde das bisher existierende Projekt java-foundations umbenennen in solution.
  3. Soll das neue Projekt mit dieser Movie-Komponente als zusätzliches Projekt dazu kommen, neben dem Projekt, was man per CLI neu erzeugt? Oder soll es das Projekt, was die TN bisher angelegt haben ersetzten?

Wir müssten hier relativ zügig voran kommen, denn die Schulung beginnt schon sehr bald.
Lieber das im Zweifelsfall mergen und für nice-to-haves noch Issues anlegen als das hier bis zur Perfektion offen zu lassen.
Wir müssen den Setup mal testen und der automatische Eclipse-Import hat seine Macken, die wir besser vor dem Training geklärt haben als im Training ;)

FYI: Eclipse auto-installation habe ich gerade angepasst: devonfw-training/java-spring-settings@4977263

import org.example.app.general.common.base.test.ApplicationComponentTest;
import org.example.app.general.common.base.test.TestUtil;
import org.example.app.moviemanagement.common.api.Genre;
import org.example.app.moviemanagement.logic.api.to.Assertions;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dieser Import kann bei mir nicht aufgelöst werden. Wenn ich z.B. eine Test-Methode should_return_fully_loaded_entity ausführe, kommt es bei mir zu Fehlern.

Copy link
Copy Markdown
Collaborator Author

@magail magail Sep 24, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dieser wird von maven generiert, wenn man in Eclipse auf "app" "Maven generate-sources" ausführt. Wenn ich allerdings dies im Terminal mache, werden sie nicht generiert.

@magail magail merged commit bae8c8a into solution-0-setup Sep 24, 2021
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