-
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: RunningRecord controller record 추가 api 추가 #52
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.
수고하셨습니다. 코드가 전반적으로 깔끔한 것 같아요👍
// FIXME: badge에 left join해서 같이 조회하기 | ||
Member member = | ||
memberRepository.findById(memberId).orElseThrow(() -> new NotFoundException(Member.class, memberId)); | ||
// TODO: 챌린지 기능 추가 후 수정 |
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.
이 부분은 스크럼때, 여기 이슈에서 논의해보면 좋을 것 같아요!
- 오늘의 챌린지 기능 구현 #39
- 필요한 기능 정리
- 테이블 구조
src/main/java/com/dnd/runus/presentation/v1/running/RunningRecordController.java
Outdated
Show resolved
Hide resolved
...ain/java/com/dnd/runus/presentation/v1/running/dto/response/RunningRecordSavingResponse.java
Outdated
Show resolved
Hide resolved
src/main/java/com/dnd/runus/application/running/RunningRecordService.java
Show resolved
Hide resolved
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.
수고 많이셨습니다 👍
🔗 이슈 연결
🚀 구현한 API
api/v1/running-records
: 혼자 뛰기 러닝 기록 추가💡 반영할 내용 및 변경 사항 요약
RunningRecordService
addRunningRecord 메서드에서는 request 값이 올바른지 검증하고 값을 저장해요RunningEmoji
에 client와 정의한 enum 이름에 맞게 수정해요🔍 리뷰 요청/참고 사항
MemberEntity
에Badge
엔티티가 없어서, fetch join을 사용할 수 없어요.그래서 JPQL을 사용해서 dto projection하려니, 문자열로 SQL을 적는거랑 다름이 없는 것 같아요.