Skip to content

Conversation

@Kammerlo
Copy link
Member

No description provided.

@Kammerlo Kammerlo requested a review from M4rc0Russ0 November 19, 2025 08:01
@Kammerlo Kammerlo self-assigned this Nov 19, 2025
…f Longs

- Updated ReportingController to return HTTP 200 OK instead of 201 CREATED for successful report creation.
- Modified ReportTemplateFieldDto and ReportTemplateResponseDto to include JsonInclude for empty lists.
- Enhanced ReportTemplateService to handle template creation and updates with appropriate error handling for existing templates.
- Updated ReportingService to use SYSTEM data mode for report generation.
- Refactored tests in ReportTemplateControllerTest and ReportingControllerTest to accommodate String IDs.
- Added new test cases for template creation and update scenarios in ReportTemplateServiceTest.
- Adjusted all relevant service and repository methods to work with String IDs instead of Longs.
M4rc0Russ0
M4rc0Russ0 previously approved these changes Nov 21, 2025
@Kammerlo Kammerlo marked this pull request as ready for review November 25, 2025 07:27
@Kammerlo Kammerlo marked this pull request as draft November 26, 2025 13:30
@Kammerlo Kammerlo marked this pull request as ready for review December 2, 2025 08:50
M4rc0Russ0
M4rc0Russ0 previously approved these changes Dec 8, 2025
M4rc0Russ0
M4rc0Russ0 previously approved these changes Dec 9, 2025
@sonarqubecloud
Copy link

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