-
Notifications
You must be signed in to change notification settings - Fork 1
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: 현재 출석 상태 조회 기능 구현 #29
Conversation
Auto labels applied based on the title of the PR. |
|
||
val monday = input.now.getMonday() | ||
val thisWeekSession = sessionGateway.findByStartTimeBetween( | ||
monday, |
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.
이거 당일이 아니라 그 주 range로 찾는 이유가 따로 있나용?
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.
원래 있던 로직 가져온거긴한데 당일 아니면 시간체크 다르게 먹어서 크게 상관없을것 같긴 합니다!
} | ||
|
||
private fun LocalDateTime.getMonday(): LocalDateTime { | ||
val dayOfWeek = this.dayOfWeek |
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.
요거 LocalDate#withDayOfWeek(MONDAY) 형태로도 꺼낼 수 있었던거같아요
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.
수고하셨습니다!
💡 기능
🔎 기타
Close #28