search

pxlit-projects / 3tiwfullstack-labs-AngelChezziPXL

3tiwfullstack-labs-3tiwfullstack-labs-3TIWFullStack2425 created by GitHub Classroom
0 stars 0 forks source link

Query employees werkt niet #1

Open AngelChezziPXL opened 1 month ago

AngelChezziPXL commented 1 month ago

@tomschuyten Dag meneer Schuyten

Hoe kan ik de employees toch toevoegen aan de db zodat ik de query werkend krijg? Momenteel staan de employees transient. DepartmentDomain

link naar de query: DepartmentRepository

Alvast bedankt.

Mvg

tomschuyten commented 1 month ago

Met Open-Feign. Opgelet, in Departement worden geen employees opgeslagen.

AngelChezziPXL commented 1 month ago

@tomschuyten Bedankt. Voorlopig ben ik nog niet zo ver. Wat kan ik doen om alsnog de service te kunnen opstarten en de testen te kunnen schrijven en runnen, zodat ik de eerste twee labs kan afwerken? Nu krijg ik altijd een compile error en kan ik niet verder.

AngelChezziPXL commented 1 month ago

@tomschuyten Dag meneer Schuyten Ik heb ondertussen Open Feign geïmplementeerd in mijn labs. Zou u kunnen kijken of wat ik doe om de employees mee te krijgen met de response van de departments correct is? Is het ok om een for loop te gebruiken of zou het beter zijn met een stream? https://github.com/pxlit-projects/3tiwfullstack-labs-AngelChezziPXL/blob/ed84291f2745ccea4e7352732e95a9b3a2d11210/backend-java/SpringCloudPXL/department-service/src/main/java/be/pxl/services/department/services/DepartmentService.java#L55C1-L62C10

Is het in de volgende code stuk eigenlijk nog nodig om andere methode te hebben van de repository of kan ik gewoon de vorige hergebruiken? https://github.com/pxlit-projects/3tiwfullstack-labs-AngelChezziPXL/blob/ed84291f2745ccea4e7352732e95a9b3a2d11210/backend-java/SpringCloudPXL/department-service/src/main/java/be/pxl/services/department/services/DepartmentService.java#L49

tomschuyten commented 4 weeks ago

Volgens dat ik kan zien sla jij ook organizations op in department. Dat klopt niet, in departement haal je via de employeeclient de employees op voor een departement. In organization doe je dit voor organizations (via employeeclient employees ophalen voor deze organization). Kijk je dat ns na?