Feat/add mentor backend

[sifterstudios]

Added model, migration and endpoints for mentor feature (personalleder).
I'm probably forgetting some endpoint that'll be needed, and would also like to add some tests to this.
Comments are very welcome as I'm pretty new to the .NET world of things! 😆

[Copilot]

Copilot reviewed 14 out of 15 changed files in this pull request and generated 1 comment.

Files not reviewed (1)

• backend/Api/appsettings.Local.json.template: Language not supported

[sigridge]

Ting ser bra ut! Men har noen kommentarer som jeg gjerne vil drøfte 😄

[Copilot]

Pull Request Overview

This PR implements backend functionality for mentor and personnel team features by adding new models, migrations, endpoints, and repository methods.

• Adds mentor capability by introducing a MentorId field on Consultant and creating a new Mentors table
• Introduces personnel team management with endpoints and associated repository methods, models, and migrations
• Updates test container initialization and docker-compose configuration for consistency

Reviewed Changes

Copilot reviewed 18 out of 20 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
docker-compose.yml	Updates port syntax to a consistent style
TestContainersFactory.cs	Refactors container setup using a generic container builder
ConsultantDbRepository.cs	Adds personnel team related repository methods
Migrations/*	Creates migrations for PersonnelTeams and Mentors tables
ApplicationContext.cs	Registers new DbSets for PersonnelTeam and PersonnelTeamByConsultant
PersonnelTeamController.cs	Adds endpoints managing personnel teams and their members
ConsultantController.cs	Adds endpoint to update personnel team association for a consultant
PersonnelTeamReadModel.cs	Introduces a read model for personnel team data

Files not reviewed (2)

• backend/Api/appsettings.Local.json.template: Language not supported
• backend/Infrastructure/Infrastructure.csproj: Language not supported

Comments suppressed due to low confidence (1)

backend/Api/Consultants/PersonnelTeamController.cs:91

• The DeletePersonnelTeam endpoint returns a Task<Task> which causes unnecessary double-awaiting. It is recommended to update the method signature to return Task directly.

public async Task<Task<IResult>> DeletePersonnelTeam([FromRoute(Name = "personnelTeamId")] int personnelTeamId, CancellationToken cancellationToken)

[astride]

Noe pirk, noen forslag, noen undringer 😊

[sifterstudios]

nå er de pushet! :D  On 30/04/2025 13:34:29, Astrid Eldhuset ***@***.***> wrote: @astride commented on this pull request. In backend/Infrastructure/Repositories/Consultants/ConsultantDbRepository.cs [#701 (comment)]:

+ }; + + var savedPersonnelTeam = await context.PersonnelTeams.AddAsync(personnelTeam, cancellationToken); + + await context.SaveChangesAsync(cancellationToken); + return savedPersonnelTeam.Entity.Id; + } + + public async Task<IResult> UpdatePersonnelTeamByConsultantId(int consultantId, int? personnelTeamId, CancellationToken cancellationToken) + { + var affectedRows = await context.PersonnelTeamByConsultants + .Where(c => c.ConsultantId == consultantId) + .ExecuteDeleteAsync(cancellationToken); + if (affectedRows == 0) + { + return Results.NotFound($"Could not find consultant with id {consultantId}");

Hmm, har du pushet oppdateringene? 👀 — Reply to this email directly, view it on GitHub [#701 (comment)], or unsubscribe [https://github.com/notifications/unsubscribe-auth/AQQM3BQXUQGDK6SMF5MR45L24CYMJAVCNFSM6AAAAAB3IDPZEWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDQMBWG42TGMBSGE]. You are receiving this because you were assigned.Message ID: ***@***.***> [bf4364ea-108d-4c30-ac1d-debf7611a411]