SZUT/ProjectManagmentTool
5
0
Fork 0
Code Issues Pull requests Releases Wiki Activity Actions

WIP: Zum Besprechen #13

Closed
SZUT-Dominik wants to merge 0 commits from just-for-talking into trunk
pull from: just-for-talking
merge into: SZUT:trunk
Conversation 44 Commits - Files changed -
SZUT-Dominik commented 2024-10-16 10:50:32 +00:00
Owner
Copy link
No description provided.
SZUT-Ole commented 2024-10-16 10:52:56 +00:00
Member
Copy link

Ich hab keine Ahnung wie ich dir über PR's schreiben soll, wenn ich nichts fertig habe. Habe mich mit Rajbir für das PMT-4er zusammengesetzt, wir kamen aber zu nichts und ich komme immer noch nicht weiter

Ich hab keine Ahnung wie ich dir über PR's schreiben soll, wenn ich nichts fertig habe. Habe mich mit Rajbir für das PMT-4er zusammengesetzt, wir kamen aber zu nichts und ich komme immer noch nicht weiter
SZUT-Dominik commented 2024-10-16 10:55:08 +00:00
Author
Owner
Copy link

Ich hab keine Ahnung wie ich dir über PR's schreiben soll, wenn ich nichts fertig habe. Habe mich mit Rajbir für das PMT-4er zusammengesetzt, wir kamen aber zu nichts und ich komme immer noch nicht weiter

womit genau, kommt ihr nicht wieter, mit der oas spec?

ich rebase euch einmal den branch auf den neusten stand.

> Ich hab keine Ahnung wie ich dir über PR's schreiben soll, wenn ich nichts fertig habe. Habe mich mit Rajbir für das PMT-4er zusammengesetzt, wir kamen aber zu nichts und ich komme immer noch nicht weiter womit genau, kommt ihr nicht wieter, mit der oas spec? ich rebase euch einmal den branch auf den neusten stand.
SZUT-Ole commented 2024-10-16 11:06:10 +00:00
Member
Copy link

Hatte mir schon vom aktuellen trunk nen neuen Branch erstellt. Komme aber einfach nicht drauf wie ich diese Mitarbeiter zu einem Projekt zuweise. Muss ich die erstmal als Objekt in der pmt.yml und den Mapper anfügen? Woher bekomme ich das ganze Objekt von einem Angestellten? Ich bin einfach gerade bisschen im Chaos versunken

Hatte mir schon vom aktuellen trunk nen neuen Branch erstellt. Komme aber einfach nicht drauf wie ich diese Mitarbeiter zu einem Projekt zuweise. Muss ich die erstmal als Objekt in der pmt.yml und den Mapper anfügen? Woher bekomme ich das ganze Objekt von einem Angestellten? Ich bin einfach gerade bisschen im Chaos versunken
SZUT-Dominik commented 2024-10-16 11:10:03 +00:00
Author
Owner
Copy link

Hatte mir schon vom aktuellen trunk nen neuen Branch erstellt. Komme aber einfach nicht drauf wie ich diese Mitarbeiter zu einem Projekt zuweise. Muss ich die erstmal als Objekt in der pmt.yml und den Mapper anfügen? Woher bekomme ich das ganze Objekt von einem Angestellten? Ich bin einfach gerade bisschen im Chaos versunken

Ich hatte die Datenbank modelle bereits angelegt, in dem branch in den ich vorbereitet hatte, für euch hies das eigenlich nur das ihr den endpunkt und dtos noch definieren müsst und dann die implementierung und tests shcrieben müsst

> Hatte mir schon vom aktuellen trunk nen neuen Branch erstellt. Komme aber einfach nicht drauf wie ich diese Mitarbeiter zu einem Projekt zuweise. Muss ich die erstmal als Objekt in der pmt.yml und den Mapper anfügen? Woher bekomme ich das ganze Objekt von einem Angestellten? Ich bin einfach gerade bisschen im Chaos versunken Ich hatte die Datenbank modelle bereits angelegt, in dem branch in den ich vorbereitet hatte, für euch hies das eigenlich nur das ihr den endpunkt und dtos noch definieren müsst und dann die implementierung und tests shcrieben müsst
SZUT-Dominik commented 2024-10-16 11:10:18 +00:00
Author
Owner
Copy link

gib mir aber nochmal ne 1/4 stunde

gib mir aber nochmal ne 1/4 stunde
SZUT-Ole commented 2024-10-16 11:11:34 +00:00
Member
Copy link

Ich mach jetzt sowieso erstmal eben mittag

Ich mach jetzt sowieso erstmal eben mittag
👍 1
SZUT-Dominik commented 2024-10-16 11:29:17 +00:00
Author
Owner
Copy link

Schritte für euch:

  1. Zieht euch den aktuellen branch zu dem Ticket von mir.
  2. Beschreibt den Endpunkt in der pmt.yml
  3. Sagt mir einmal bescheid und ich sage ob ihr noch was am endpunkt anpassen müsst. (allses schriftlich)
  4. Genneriert und implementiert den neuen Endpunkt im Controller. Nachfragen dazu zu mir dann. (scrhiftlich)
    1. Schaut euch gerne das ann was ich bereits im Branch gemacht habe, aber ihr erstellt zum zuweisen einfach nur ein Alllocation Objeckt und speichert es in der Datenbank. auf deutsch ist es ja ein "zuweisungs objekt". Siehe https://git.euph.dev/SZUT/ProjectManagmentTool/wiki/Datastructure
  5. Schreibt die Unittests
  6. macht die PR auf und fügt mich als reviewer hinzu
  7. schiebt das ticket in QS und weist es mit zu.
  8. weitere schritte werden sich dann geben
Schritte für euch: 1. Zieht euch den **aktuellen** branch zu dem Ticket von mir. 1. Beschreibt den Endpunkt in der `pmt.yml` 1. Sagt mir einmal bescheid und ich sage ob ihr noch was am endpunkt anpassen müsst. **(allses schriftlich)** 1. Genneriert und implementiert den neuen Endpunkt im Controller. Nachfragen dazu zu mir dann. **(scrhiftlich)** 1. Schaut euch gerne das ann was ich bereits im Branch gemacht habe, aber ihr erstellt zum zuweisen einfach nur ein `Alllocation` Objeckt und speichert es in der Datenbank. auf deutsch ist es ja ein "zuweisungs objekt". Siehe https://git.euph.dev/SZUT/ProjectManagmentTool/wiki/Datastructure 1. Schreibt die Unittests 1. macht die PR auf **und** fügt mich als reviewer hinzu 1. schiebt das ticket in QS und weist es mit zu. 1. weitere schritte werden sich dann geben
SZUT-Dominik changed title from Zum Besprechen to WIP: Zum Besprechen 2024-10-16 11:35:57 +00:00
SZUT-Ole commented 2024-10-16 13:07:08 +00:00
Member
Copy link

Habe ein neue Push gemacht wo ich angefangen habe den Endpunkt zu definieren. Ich finde aber leider nicht raus wie ich das mache, wenn ich 2 Variablen übergeben bekomme. Ich brauche ja sowohl die projectId als auch die employeeId für die zuweisung. (er will irgendwie keinen Link Hinzufügen sonst hätte ich dir die commit hier verlinkt.)

Habe ein neue Push gemacht wo ich angefangen habe den Endpunkt zu definieren. Ich finde aber leider nicht raus wie ich das mache, wenn ich 2 Variablen übergeben bekomme. Ich brauche ja sowohl die projectId als auch die employeeId für die zuweisung. (er will irgendwie keinen Link Hinzufügen sonst hätte ich dir die commit hier verlinkt.)
SZUT-Dominik commented 2024-10-16 13:07:53 +00:00
Author
Owner
Copy link

@SZUT-Ole bei dem hinzufügen eines Mitarbeiters würde ich die Mitarbeiter ID, so wie die ID der Qualifikation, die als sein Rolle gilt, in dem Request body entgegen nehmen, das wäre REST standard

@SZUT-Ole bei dem hinzufügen eines Mitarbeiters würde ich die Mitarbeiter ID, so wie die ID der Qualifikation, die als sein Rolle gilt, in dem Request body entgegen nehmen, das wäre REST standard
SZUT-Dominik commented 2024-10-16 13:09:17 +00:00
Author
Owner
Copy link

Siehe die create Projekt Definition in der doc, wie es geht

Siehe die create Projekt Definition in der doc, wie es geht
SZUT-Dominik commented 2024-10-16 13:13:49 +00:00
Author
Owner
Copy link

Commits verlinkst du in dem du einfach nur den hash der commits einfügst mit lehrzeilen herum ed7245e367

Commits verlinkst du in dem du einfach nur den hash der commits einfügst mit lehrzeilen herum ed7245e36745dfea37edb5be7e915c802e2bec70
SZUT-Ole commented 2024-10-16 13:16:54 +00:00
Member
Copy link

Haben wir denn schon ein Feld für Qualifikationen im Project?

Dann brauchen wir also 3 Parameter die wir mitgeben? Die projectId brauchen wir doch um zu wissen auf welches Project wir posten müssen oder nicht? Mit der doc zu createProject meinst du das post auf dem project Endpunkt?

Haben wir denn schon ein Feld für Qualifikationen im Project? Dann brauchen wir also 3 Parameter die wir mitgeben? Die projectId brauchen wir doch um zu wissen auf welches Project wir posten müssen oder nicht? Mit der doc zu createProject meinst du das post auf dem project Endpunkt?
SZUT-Dominik commented 2024-10-16 13:18:44 +00:00
Author
Owner
Copy link

Projekt ID in der URl,
Mitarbeiter und qualifikations ID in den body.

Und speichern tust du ein neues Allocation object in der Datenbank, das habe ich ja auch schon angelegt

Projekt ID in der URl, Mitarbeiter und qualifikations ID in den body. Und speichern tust du ein neues Allocation object in der Datenbank, das habe ich ja auch schon angelegt
SZUT-Dominik commented 2024-10-16 13:19:47 +00:00
Author
Owner
Copy link

Die Allocation hat die Kombination aus Projekt und Mitarbeiter ID (beide forign keys) als sein primary key

Die Allocation hat die Kombination aus Projekt und Mitarbeiter ID (beide forign keys) als sein primary key
SZUT-Ole commented 2024-10-16 13:22:59 +00:00
Member
Copy link

Ich weiss in etwa was du meinst, aber die Syntax ist mir da nicht geläufig. Der Content ist beim post auf project ja auch ganz anders und verlinkt nur aufs CreateProjectDTO.

Ich bin für heute aber jetzt gleich auch erstmal raus und würde morgen weiter probieren.

Ich weiss in etwa was du meinst, aber die Syntax ist mir da nicht geläufig. Der Content ist beim post auf project ja auch ganz anders und verlinkt nur aufs CreateProjectDTO. Ich bin für heute aber jetzt gleich auch erstmal raus und würde morgen weiter probieren.
SZUT-Dominik commented 2024-10-16 13:29:33 +00:00
Author
Owner
Copy link

Genau, der content verlinkt, also Legst du ein AddEmployeeDTO an, welches den Employee und die Qualifikations ID als Felder hat, was ja beides longs in java, also int64 vom Format sind.

Genau, der content verlinkt, also Legst du ein `AddEmployeeDTO` an, welches den Employee und die Qualifikations ID als Felder hat, was ja beides longs in java, also int64 vom Format sind.
👍 1
SZUT-Ole commented 2024-10-17 09:12:50 +00:00
Member
Copy link

So, ich hab das DTO angelegt und den Endpunkt beschrieben, kannst du dir ja anschauen wenn du zeit hast.

Warum wird das neue DTO eigentlich nicht commited, wenn die anderen DTO's doch auch hinzugefügt wurden?

So, ich hab das DTO angelegt und den Endpunkt beschrieben, kannst du dir ja anschauen wenn du zeit hast. Warum wird das neue DTO eigentlich nicht commited, wenn die anderen DTO's doch auch hinzugefügt wurden?
SZUT-Dominik commented 2024-10-17 09:18:24 +00:00
Author
Owner
Copy link

Dtos sind doch nicht commited, commited sind doch nur models

Dtos sind doch nicht commited, commited sind doch nur models
SZUT-Dominik commented 2024-10-17 09:21:29 +00:00
Author
Owner
Copy link

Du hast den request body mit dem path gemixt, du musst requestBody nutzen

Du hast den request body mit dem path gemixt, du musst `requestBody` nutzen
SZUT-Ole commented 2024-10-17 09:27:30 +00:00
Member
Copy link

aber der path ist doch notwendig wenn wir zu einem speziellen project einen mitarbeiter hinzufügen wollen oder nicht?
Das einzige andere mal wo wir den request body benutzen ist beim createProject und da brauchen wir ja gar keine id im path mitgeben.

Wie kann ich das denn sonst schreiben?

aber der path ist doch notwendig wenn wir zu einem speziellen project einen mitarbeiter hinzufügen wollen oder nicht? Das einzige andere mal wo wir den request body benutzen ist beim createProject und da brauchen wir ja gar keine id im path mitgeben. Wie kann ich das denn sonst schreiben?
SZUT-Dominik commented 2024-10-17 10:01:04 +00:00
Author
Owner
Copy link

Mach beides auf einmal rein

Mach beides auf einmal rein
SZUT-Dominik commented 2024-10-17 10:01:16 +00:00
Author
Owner
Copy link

Wichtig sind die einrückungen

Wichtig sind die einrückungen
SZUT-Ole commented 2024-10-17 11:07:11 +00:00
Member
Copy link

Hab das mal geupdatet. So meintest du, oder?

Hab das mal geupdatet. So meintest du, oder?
SZUT-Dominik commented 2024-10-17 11:22:11 +00:00
Author
Owner
Copy link

Ja so, du hast nur bei der path variable

schema: 
  type: integer
  format: int64
required: true

Vergessen

Ja so, du hast nur bei der path variable ```yml schema: type: integer format: int64 required: true ``` Vergessen
SZUT-Dominik commented 2024-10-17 11:22:39 +00:00
Author
Owner
Copy link

Schau aber nochmal auf die anderen, ob ich es richtig geschrieben habe.

Schau aber nochmal auf die anderen, ob ich es richtig geschrieben habe.
SZUT-Dominik commented 2024-10-17 11:23:15 +00:00
Author
Owner
Copy link

Und versuche lokal den validerungs job für das pmt auszuführen, aktuell schlägt der in der Pipeline bei dir fehl

Und versuche lokal den validerungs job für das pmt auszuführen, aktuell schlägt der in der Pipeline bei dir fehl
SZUT-Dominik commented 2024-10-17 11:29:46 +00:00
Author
Owner
Copy link

500 und 401 kannst du aucnoch zu dem Endpunkt hinzufügen

500 und 401 kannst du aucnoch zu dem Endpunkt hinzufügen
SZUT-Ole commented 2024-10-17 11:32:23 +00:00
Member
Copy link

Alles klar, perfekt, hab ich angepasst und Validierung lief auch durch mit grünem Haken.

Alles klar, perfekt, hab ich angepasst und Validierung lief auch durch mit grünem Haken.
👍 1
SZUT-Dominik commented 2024-10-17 11:45:47 +00:00
Author
Owner
Copy link

Dann kannst du jetzt mit implementieren anfangen

Dann kannst du jetzt mit implementieren anfangen
SZUT-Ole commented 2024-10-17 11:50:57 +00:00
Member
Copy link

Ich werde es versuchen, auch wenn es noch nicht komplett verständlich für mich ist. Vor allem die try/catch Blöcke im createProject ist für mich sehr schwierig

Ich werde es versuchen, auch wenn es noch nicht komplett verständlich für mich ist. Vor allem die try/catch Blöcke im createProject ist für mich sehr schwierig
SZUT-Dominik commented 2024-10-17 11:57:09 +00:00
Author
Owner
Copy link

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

Jo, ich mache jz min. 2 Stunden mittag, habe Gartenarbeit gemacht bei meinem Vater
SZUT-Ole commented 2024-10-17 12:07:37 +00:00
Member
Copy link

Alles gut, mal schauen wie viel und wie schnell ich was schaffe. Bin gesundheitlich leider gerade nicht ganz fit. hoffentlich wird's nicht noch schlimmer. Im Homeoffice ist das zum glück nicht so stark belastend wie in einer Gruppenarbeit vor Ort.

Alles gut, mal schauen wie viel und wie schnell ich was schaffe. Bin gesundheitlich leider gerade nicht ganz fit. hoffentlich wird's nicht noch schlimmer. Im Homeoffice ist das zum glück nicht so stark belastend wie in einer Gruppenarbeit vor Ort.
SZUT-Dominik commented 2024-10-17 12:29:25 +00:00
Author
Owner
Copy link

Joa, mach gern etwas ruhe

Joa, mach gern etwas ruhe
SZUT-Ole commented 2024-10-17 13:19:23 +00:00
Member
Copy link

Für den Fall, dass ich morgen ausfalle und Rajbir für mich weitermacht:
Aktuell hat er auf dem Branch den Blauen und den Grünen Pfeil, aber mit fetch und pull bekommt er trotzdem nicht meinen Stand. Vielleicht kannst du ihm da ja helfen.

Für den Fall, dass ich morgen ausfalle und Rajbir für mich weitermacht: Aktuell hat er auf dem Branch den Blauen und den Grünen Pfeil, aber mit fetch und pull bekommt er trotzdem nicht meinen Stand. Vielleicht kannst du ihm da ja helfen.
SZUT-Rajbir commented 2024-10-17 13:20:44 +00:00
Member
Copy link

Um sicher zugehen:

Wie hole ich Oles Stand vom Ticket?

  • mit git fetch git pull?

ich hab das gemacht und habe diesen grünen und bauen Pfeil bekommen. Und habe nicht den Stand von ihm.

Um sicher zugehen: Wie hole ich Oles Stand vom Ticket? - mit git fetch git pull? ich hab das gemacht und habe diesen grünen und bauen Pfeil bekommen. Und habe nicht den Stand von ihm.
SZUT-Rajbir closed this pull request 2024-10-17 13:20:44 +00:00
SZUT-Rajbir reopened this pull request 2024-10-17 13:24:10 +00:00
SZUT-Dominik commented 2024-10-17 13:34:00 +00:00
Author
Owner
Copy link
  • git checkout story/PMT-X-der-branch-name
  • git fetch
  • git reset --hard origin/story/PMT-X-der-name
- git checkout story/PMT-X-der-branch-name - git fetch - git reset --hard origin/story/PMT-X-der-name
👍 1
SZUT-Rajbir commented 2024-10-18 08:40:20 +00:00
Member
Copy link

@SZUT-Dominik bei im ApiController wird die "public class ApiController implements DefaultApi" als rot unterstrichen, intelliji sagst, dass ->Class 'ApiController' must either be declared abstract or implement abstract method 'addEmployee(Long, AddEmployeeDTO)' in 'DefaultApi'. Soll ich die methods implementieren ?

@SZUT-Dominik bei im ApiController wird die "public class ApiController implements DefaultApi" als rot unterstrichen, intelliji sagst, dass ->Class 'ApiController' must either be declared abstract or implement abstract method 'addEmployee(Long, AddEmployeeDTO)' in 'DefaultApi'. Soll ich die methods implementieren ?
SZUT-Rajbir commented 2024-10-18 09:22:36 +00:00
Member
Copy link

@SZUT-Dominik bei im ApiController wird die "public class ApiController implements DefaultApi" als rot unterstrichen, intelliji sagst, dass ->Class 'ApiController' must either be declared abstract or implement abstract method 'addEmployee(Long, AddEmployeeDTO)' in 'DefaultApi'. Soll ich die methods implementieren ?

ist geklärt

> @SZUT-Dominik bei im ApiController wird die "public class ApiController implements DefaultApi" als rot unterstrichen, intelliji sagst, dass ->Class 'ApiController' must either be declared abstract or implement abstract method 'addEmployee(Long, AddEmployeeDTO)' in 'DefaultApi'. Soll ich die methods implementieren ? ist geklärt
SZUT-Rajbir commented 2024-10-18 09:46:51 +00:00
Member
Copy link

@SZUT-Dominik ich kommen an den Punkt "Mitarbeiter, die schon verplant sind, können nicht zugeordnet werden → HTTP 409" nicht weiter.

Wie kann überprüft werden, ob der Mitarbeiter verplant ist ?

  • hat das was mit dem start, planned end und real end zu tun ?
@SZUT-Dominik ich kommen an den Punkt "Mitarbeiter, die schon verplant sind, können nicht zugeordnet werden → HTTP 409" nicht weiter. Wie kann überprüft werden, ob der Mitarbeiter verplant ist ? - hat das was mit dem start, planned end und real end zu tun ?
SZUT-Dominik commented 2024-10-18 10:31:06 +00:00
Author
Owner
Copy link

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 Kollidieren würde, wenn ja den 409 werfen
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 Kollidieren würde, wenn ja den 409 werfen
SZUT-Rajbir commented 2024-10-18 15:23:05 +00:00
Member
Copy link

@SZUT-Dominik du kann dir meinen aktuellen Stand anschauen. Ich muss aber die Test noch schreiben.

@SZUT-Dominik du kann dir meinen aktuellen Stand anschauen. Ich muss aber die Test noch schreiben.
SZUT-Dominik commented 2024-10-18 15:37:28 +00:00
Author
Owner
Copy link

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

Mache ich gleich, muss gerade noch was erledigen, dir einen schönen Feierabend., ich nehme mir etwas Zeit dafür
SZUT-Dominik closed this pull request 2024-10-18 20:24:34 +00:00
SZUT-Dominik commented 2024-10-18 20:25:17 +00:00
Author
Owner
Copy link

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

ich habe nur den branch den ich hierfür gelöscht, können trozdem weiter schreiben hier
SZUT-Dominik commented 2024-10-18 21:33:54 +00:00
Author
Owner
Copy link

schaut bitte in !14 habe dort ein erstes code review schomal gemacht

schaut bitte in !14 habe dort ein erstes code review schomal gemacht
All checks were successful
Quality Check / Validate OAS (push) Successful in 42s
Required
Details
Quality Check / Linting (push) Successful in 1m53s
Required
Details
Quality Check / Testing (push) Successful in 2m11s
Required
Details
Quality Check / Static Analysis (push) Successful in 2m15s
Required
Details
Quality Check / Validate OAS (pull_request) Successful in 33s
Required
Details
Quality Check / Linting (pull_request) Successful in 1m9s
Required
Details
Quality Check / Testing (pull_request) Successful in 1m14s
Required
Details
Quality Check / Static Analysis (pull_request) Successful in 1m16s
Required
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
  
type
Bug

  
type
Story

  
type
Task

No labels
type
Bug
type
Story
type
Task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: SZUT/ProjectManagmentTool#13
No description provided.
Delete branch "just-for-talking"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?