Skip to content

Remove break time fields from work policy model and related classes#10

Merged
subsub97 merged 1 commit intomainfrom
remove-break-time
Feb 7, 2026
Merged

Remove break time fields from work policy model and related classes#10
subsub97 merged 1 commit intomainfrom
remove-break-time

Conversation

@jeyongsong
Copy link
Member

This pull request removes the break time fields (breakStartTime and breakEndTime) from the work policy data model and related service logic. The changes simplify the handling of work policies by eliminating all references to break times from entities, DTOs, service methods, and validation logic.

Data model and DTO changes:

  • Removed the breakStartTime and breakEndTime fields from the WorkPolicyVersion entity, as well as from the WorkPolicyUpsertRequest and WorkPolicyResponse data classes. [1] [2] [3]

Service logic updates:

  • Eliminated all usage of breakStartTime and breakEndTime in the WorkPolicyService and OnboardingStatusService, including their handling in constructors, updates, and response mapping. [1] [2] [3]
  • Removed validation logic related to break times from the validateRequest method in WorkPolicyService.

@github-actions
Copy link

github-actions bot commented Feb 7, 2026

Test Results

8 tests   8 ✅  0s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit a1627e0.

Copy link
Collaborator

@subsub97 subsub97 left a comment

Choose a reason for hiding this comment

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

Thx 👍

@subsub97 subsub97 merged commit cf5af31 into main Feb 7, 2026
2 checks passed
@subsub97 subsub97 deleted the remove-break-time branch February 7, 2026 14:21
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.

2 participants