Review to 박성현

[Horaiz-UQ]

성현님 구현 코드에 대한 코드 리뷰입니다.
연차/당직을 담당해서 부족하지만 연차/당직 로직 부분에 대한 코드 리뷰를 하겠습니다!

1. 연차 신청을 할 때 예외처리를 추가했으면 합니다

• 연차 신청일이 종료일보다 뒤에 있을 때 따로 예외처리 해주는 부분이 없는 것 같습니다.

• 연차를 신청할때 신청연차와 남은 연차를 계산하시는데 실제 반영은 관리자 쪽에서 하시는 것 같은데, 연차 사용 일 처리가 승인되었을 때만 하시면 실제 반영은 되어있지 않으니 사용자가 무제한으로 신청이 가능할 것 같습니다.

뭔가 부족한 리뷰지만 프로젝트 고생하셨습니다!!

[seonghye0n]

리뷰 감사드립니다! 🙇
연차 신청일이 종료일보다 뒤에 있을 때 예외처리는 @AfterStartDate 라는 커스텀 어노테이션을 생성해서 적용했습니다!

무제한 신청 관련해서는 내년 연차를 땡겨쓰는 개념으로 해서 스케쥴러는 리팩토링 기간에 추가 예정이여서 -로 계속 신청되고 내년에 스케쥴러로 연차 셋팅을 다시 할 때, 차감된 갯수를 적용할 예정입니다!