-
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
[Feature] 리마인드 관련 API 개발 #26
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.
수고하셨습니다..! 충돌만 해결하고 푸시알림만 성공하면 순조롭겠네용!
import jakarta.persistence.Converter; | ||
import java.util.ArrayList; | ||
|
||
@Converter |
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.
converter라는 어노테이션을 써본적없었는데 이렇게 활용할 수 있군용..!
.collect(Collectors.toList()); | ||
|
||
List<WaitingTimerDto> waitingTimerList = reminders.stream() | ||
.filter(reminder -> !isCompletedTimer(reminder)) |
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.
오 필터까지 깔끔하게 적용한 것 같네용!! 좋습니당!
🚩 관련 이슈
📋 구현 기능 명세
📌 PR Point
무슨 이유로 어떻게 코드를 변경했는지
타이머의 리마인드 요일을 Mon, Tue 글자로 구분하는 형태에서 1->월요일, 2->화요일 ... 숫자로 구분하는 것으로 바꿨습니다!
숫자로 구분하는것이 현재날짜와 비교하기도 편하고 매번 문자열을 바꿔서 비교하기도 불편하여서....
어떤 부분에 리뷰어가 집중해야 하는지
TimerService 클래스 내에 타이머 메인페이지 조회하는 getTimerPage()안에 코드 깔끔하게 하려고했는데 괜찮은지....
개발하면서 어떤 점이 궁금했는지
📸 결과물 스크린샷
🛠️ 테스트
🚀 API Endpoint