SZUT-Ole
  • Joined on 2024-03-12
SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 11:50:57 +00:00
WIP: Zum Besprechen

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

e1fc61dfdd PMT-4: Code fix #3
SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 11:32:23 +00:00
WIP: Zum Besprechen

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

ffae4e493b PMT-4: Code fix #2
SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 11:07:11 +00:00
WIP: Zum Besprechen

Hab das mal geupdatet. So meintest du, oder?

SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 09:27:30 +00:00
WIP: Zum Besprechen

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 createProje…

SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-17 09:12:50 +00:00
WIP: Zum Besprechen

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…

2a35474ed3 PMT-4: Add AddEmpployeeDTO and endpoint specification
SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-16 13:22:59 +00:00
WIP: Zum Besprechen

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…

SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-16 13:16:54 +00:00
WIP: Zum Besprechen

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…

SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-16 13:07:08 +00:00
WIP: Zum Besprechen

Habe einen neuen 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…

ed7245e367 PMT-4: Define new endpoint
SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-16 11:11:34 +00:00
WIP: Zum Besprechen

Ich mach jetzt sowieso erstmal eben mittag

SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-16 11:06:10 +00:00
WIP: Zum Besprechen

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…

SZUT-Ole commented on pull request SZUT/ProjectManagmentTool#13 2024-10-16 10:52:56 +00:00
WIP: Zum Besprechen

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…

SZUT-Ole 2024-10-15 08:15:23 +00:00
SZUT-Ole pushed to trunk at SZUT/ProjectManagmentTool 2024-10-15 08:15:23 +00:00
d38c5a1084 Merge pull request 'PMT-16: Projekt Anlegen' (!12) from story/PMT-16-projekt-anlegen into trunk
50d889820d PMT-16: Cleanup Test Cases
a1710ef2fa PMT-16: Edit and create more test cases
37048dab6e PMT-16: Testing Create Project Endpoint
7fb6b7524f PMT-16: Update Manual Validation Method of Projects to Include the builtin.
Compare 11 commits »
SZUT-Ole pushed to story/PMT-16-projekt-anlegen at SZUT/ProjectManagmentTool 2024-10-14 09:07:51 +00:00
03e33d654e PMT-16: Cleanup Test Cases
SZUT-Ole pushed to story/PMT-16-projekt-anlegen at SZUT/ProjectManagmentTool 2024-10-14 08:59:39 +00:00
499d1279d1 PMT-16: Edit and create more test cases