Skip to content

Conversation

@sobkowiak
Copy link

wstępna wersja API.

  • Brakuje jeszcze testów.
  • mappery bez MapStruct z powodu problemów z Optional

@sobkowiak sobkowiak requested review from Copilot and piterhub and removed request for Copilot March 27, 2025 23:18
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces an initial implementation of the API for appointment booking by defining mappers and controllers for treatments and appointments. Key changes include:

  • New API mappers for translating between transfer objects and internal models.
  • Controllers for handling treatment and appointment endpoints.
  • A configuration class to wire the new mappers as Spring beans.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
src/main/java/com/capgemini/training/appointmentbooking/services/mapper/TreatmentApiMapper.java Introduces mapping logic for treatment-related objects.
src/main/java/com/capgemini/training/appointmentbooking/services/mapper/AppointmentApiMapper.java Adds conversion functions for appointment-related data.
src/main/java/com/capgemini/training/appointmentbooking/services/impl/TreatmentsApiController.java Implements treatment API endpoints.
src/main/java/com/capgemini/training/appointmentbooking/services/impl/AppointmentsApiController.java Implements appointment API endpoints.
src/main/java/com/capgemini/training/appointmentbooking/services/config/ServicesMappingConfiguration.java Configures mapper beans for dependency injection.

@sobkowiak sobkowiak self-assigned this Mar 27, 2025
@sobkowiak sobkowiak force-pushed the feature/4-services-implement-api branch from b96c61d to 4cf81ae Compare March 31, 2025 08:31
@sobkowiak sobkowiak force-pushed the feature/4-services branch from bba3df4 to e25fee4 Compare March 31, 2025 08:32
Copy link
Contributor

@piterhub piterhub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testy chcesz zrobić w osobnym PR?

@sobkowiak sobkowiak merged commit 5bedab7 into feature/4-services Apr 1, 2025
3 checks passed
@sobkowiak sobkowiak deleted the feature/4-services-implement-api branch April 1, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants