-
Notifications
You must be signed in to change notification settings - Fork 0
Development: Refine user data export
#1742
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
@sachmii Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
@sachmii Test coverage has been automatically updated in the PR description. |
|
@sachmii Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
Abi107717
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for implementing this 😊Code LGTM, Just small comments, is there are a reason, that you are using NON_NULL instead of NON_EMPTY?
src/main/java/de/tum/cit/aet/core/dto/exportdata/IntervieweeExportDTO.java
Outdated
Show resolved
Hide resolved
src/main/java/de/tum/cit/aet/core/dto/exportdata/InterviewProcessExportDTO.java
Outdated
Show resolved
Hide resolved
src/main/java/de/tum/cit/aet/core/dto/exportdata/InterviewSlotExportDTO.java
Outdated
Show resolved
Hide resolved
…t/1695-add-interview-data-to-user-data-export
|
@sachmii Test coverage has been automatically updated in the PR description. |
|
🤖 No OpenAPI or client changes needed. |
|
@sachmii Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
Checklist
General
Server
Motivation and Context
Closes #1695, needed to add interview data to user data export/
Description
Steps for Testing
Prerequisites:
Review Progress
Code Review
Manual Tests
Test Coverage
Warning: Server tests failed. Coverage could not be fully measured. Please check the workflow logs.
Last updated: 2026-01-22 17:55:55 UTC