Dominik Säume SZUT-Dominik
  • Joined on 2024-04-02
SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

Ich habe nochmal nachgeschaut, ein Not Acceptable ist was anderes und hat mit der Browser aushandlung vom Content Format zu tuhen.

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

du kannst einfach direkt repository.existsById(id) nutzen wenn du nichtmer auf as Projekt zugreifen möchtest danach, anosnten bitte direkt nach dem If check aus dem Optional rausziehen in eine neue Variable

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

Einmalige methode zum enscheiden von etwas hat hier nix zu suchen, gehört inline

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

Newline bitte wieder an das Ende tuhen.

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

Der Happy Path sollte möglichsts zu letzt kommen, du hast viel zu viel im try catch

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

Raw SQL, bitte die features von Spring Repositories direkt nutzen, siehe andere Repository Klassen

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

Newline um eine Logische Trennung zu machen

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#14 2024-10-18 21:26:37 +00:00
PMT-4: Mitarbeiter zu einem projekt hinzufügen

Ich habe nochmal nachgeschaut, ein Not Acceptable ist was anderes und hat mit der Browser aushandlung vom Content Format zu tuhen.

SZUT-Dominik created pull request SZUT/ProjectManagmentTool#14 2024-10-18 20:30:39 +00:00
WIP: story/PMT-4-mitarbeiter-zu-einem-projekt
SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#13 2024-10-18 20:25:17 +00:00
WIP: Zum Besprechen

ich habe nur den branch den ich hierfür gelöscht, können trozdem weiter schreiben hier

SZUT-Dominik closed pull request SZUT/ProjectManagmentTool#13 2024-10-18 20:24:34 +00:00
WIP: Zum Besprechen
SZUT-Dominik deleted branch just-for-talking from SZUT/ProjectManagmentTool 2024-10-18 20:24:34 +00:00
SZUT-Dominik deleted branch story/PMT-15-projekte-loeschen from SZUT/ProjectManagmentTool 2024-10-18 20:21:25 +00:00
SZUT-Dominik deleted branch story/PMT-16-projekt-anlegen from SZUT/ProjectManagmentTool 2024-10-18 20:21:20 +00:00
SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#13 2024-10-18 15:37:28 +00:00
WIP: Zum Besprechen

Mache ich gleich, muss gerade noch was erledigen, dir einen schönen Feierabend., ich nehme mir etwas Zeit dafür

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#13 2024-10-18 10:31:06 +00:00
WIP: Zum Besprechen

Ja,

  1. Alle Allocation einträge eines Employees ziehen (dafür muss I'm repository eine neue Funktion hinzugefügt werden
  2. Vergleichen ob eines der Zugewiesenen Projekte mit dem Neuen…
SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 13:34:00 +00:00
WIP: Zum Besprechen
  • git checkout story/PMT-X-der-branch-name
  • git fetch
  • git reset --hard origin/story/PMT-X-der-name
SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 12:29:25 +00:00
WIP: Zum Besprechen

Joa, mach gern etwas ruhe

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 11:57:09 +00:00
WIP: Zum Besprechen

Jo, ich mache jz min. 2 Stunden mittag, habe Gartenarbeit gemacht bei meinem Vater

SZUT-Dominik commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 11:45:47 +00:00
WIP: Zum Besprechen

Dann kannst du jetzt mit implementieren anfangen