Skip to content
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.

Sign up for GitHub

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

Jump to bottom

직원 휴무일 변경 시 충돌 위험 #312

Open
khcho0125 opened this issue Feb 11, 2023 · 0 comments
Open

직원 휴무일 변경 시 충돌 위험 #312

khcho0125 opened this issue Feb 11, 2023 · 0 comments
Assignees
@khcho0125
Labels
bug 버그가 발생한 경우 need discussion 논의가 필요한 경우 refactor 코드 리팩토링 할 경우

Comments

@khcho0125
Copy link
Member

AS-IS

1. afterDate와 beforeDate가 같을 시 PK 중복 충돌의 위험이 있습니다.

ChangeEmployeeHolidayUseCase - line 44 ~ 47

2. 같은 주 내에서 휴무일 변경 시 휴무일 제한에 걸릴 가능성이 있어 제대로 된 동작을 할 수 없습니다.

ChangeEmployeeHolidayUseCase - line 45
Holiday - line 75 ~ 79

TO-BE

  1. 메서드 순서 변경

  2. afterDate와 beforeDate Validation

  3. 별도의 메서드 사용

Additional

No Response
@khcho0125 khcho0125 added bug 버그가 발생한 경우 need discussion 논의가 필요한 경우 refactor 코드 리팩토링 할 경우 labels Feb 11, 2023
@khcho0125 khcho0125 self-assigned this Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@khcho0125 khcho0125

Labels
bug 버그가 발생한 경우 need discussion 논의가 필요한 경우 refactor 코드 리팩토링 할 경우
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@khcho0125