-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat : 퀘스트 목표 변경 기능을 구현한다 #25 #59
Conversation
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.
엄청 복잡한 기능이었는데 정말 잘 짜신거 같아요 수고하셨습니다!!
} | ||
|
||
@Scheduled(cron = "0 0 0 * * ?") // 매일 0시 0분 0초에 실행 | ||
@Scheduled(cron = "30 0 0 * * ?") // 매일 0시 0분 30초에 실행 |
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.
오 세심하네요
case "QUIZ" -> quest.updateQuizCompleted(); | ||
case "CONCEPT" -> quest.updateConceptCompleted(); | ||
case "QUIZ" -> quest.updateQuizCompletedCount(); | ||
case "CONCEPT" -> quest.updateConceptCompletedCount(); | ||
case "ARTICLE" -> quest.updateArticleCompletedCount(); | ||
default -> throw new UserException(INVALID_QUEST_TYPE); | ||
} | ||
|
||
// 퀘스트 3개 완료 시 출석 완료 처리 |
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.
여기 주석 변경해야 하는거 맞죠?
@@ -113,6 +124,8 @@ public void createAttendance(User user) { | |||
questRepository.save(Quest.builder().user(user).lastUpdatedDate(today).build()); | |||
attendanceLogRepository.save( | |||
AttendanceLog.builder().date(today).isAttended(false).attendance(attendance).build()); | |||
userGoalRepository.save( | |||
UserGoal.builder().user(user).quizGoal(1).articleGoal(3).conceptGoal(1).build()); |
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.
이게 사용자 목표 기본 설정값인거가요?
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.
수고하셨습니다!!
#️⃣연관된 이슈
📝작업 내용
💬리뷰 요구사항(선택)