-
Notifications
You must be signed in to change notification settings - Fork 1
[FEATURE] 모집글 스케쥴링 #297
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
[FEATURE] 모집글 스케쥴링 #297
Conversation
m-a-king
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.
수고하셨습니다.
디스코드 소통 내용만 봐도 상당히 검증 작업이 많이 필요해보였는데, 깔끔하게 구현해두셔서 편히 읽었습니다~
...main/java/com/somemore/domains/recruitboard/controller/RecruitBoardCommandApiController.java
Outdated
Show resolved
Hide resolved
...main/java/com/somemore/domains/recruitboard/scheduler/RecruitBoardStatusUpdateScheduler.java
Show resolved
Hide resolved
...main/java/com/somemore/domains/recruitboard/scheduler/RecruitBoardStatusUpdateScheduler.java
Outdated
Show resolved
Hide resolved
.../java/com/somemore/domains/recruitboard/scheduler/RecruitBoardStatusUpdateSchedulerTest.java
Show resolved
Hide resolved
...test/java/com/somemore/domains/recruitboard/service/validator/RecruitBoardValidatorTest.java
Outdated
Show resolved
Hide resolved
...main/java/com/somemore/domains/recruitboard/scheduler/RecruitBoardStatusUpdateScheduler.java
Outdated
Show resolved
Hide resolved
- RecruitStatus 기본 값 제거, 생성자 파라미터 추가 - 검증 로직 제거, 비지니스 로직 추가
- RecruitStatus 기본 값 제거, 생성자 파라미터 추가 - 검증 로직 제거, 비지니스 로직 추가
- 봉사 시작/종료 일시 수정시, 생성일 하루 이후부터 가능하도록 검증
- 봉사 시작/종료 일시 수정시, 생성일 하루 이후부터 가능하도록 검증
- 모집글 수정시 시간에 따른 검증 일괄 추가 - 봉사 시작 일시 전날까지 수정 가능하도록 검증
- 모집글 수정시 시간에 따른 검증 일괄 추가 - 봉사 시작 일시 전날까지 수정 가능하도록 검증
- 봉사 시작일 기준 모집완료(CLOSED) 변경 - 봉사 종료일 기준 완료(COMPLETED) 변경
- 봉사 시작일 기준 모집완료(CLOSED) 변경 - 봉사 종료일 기준 완료(COMPLETED) 변경
- 최대 시도, 주기 명시적으로 변경
e3c6562 to
97c9e20
Compare
ayoung-dev
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.
고생하셨습니다!👍🏻
- Service 계층에서 Clock 사용
- Service 계층에서 Clock 사용
- between()으로 변경
- between()으로 변경
|



resolved:
📌 과제 설명
👩💻 요구 사항과 구현 내용
✅ PR 포인트 & 궁금한 점