-
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] 회원 데일리 루틴 추가 #34
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.
GPT도 찾지 못한 코드의 문제... 이번에는 테스트코드를 먼저 작성하고 코드 작성하셨는데 작업 방식을 바꾼 이유가 있나요?? 저번까지는 작업 후 테스트 코드를 짜신 거 같아서!
@csb9427 기능 코드 작성 후 테스트 코드 나중에 짠 거 맞아요! 모든 작업 완료 후 커밋 순서를 임의로 하다보니 이렇게 된 것 같네요..! |
답변 감사합니당~~ 제가 모르는 심오한 뜻이 있을까 해서... ㅎㅎ |
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.
GOAT
public MemberDailyRoutineResponse createMemberDailyRoutine(long memberId, MemberDailyRoutineRequest request) { | ||
val member = findMember(memberId); | ||
val routine = findRoutine(request.routineId()); | ||
val memberRoutine = new MemberDailyRoutine(member, routine); |
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.
member, routine에 findMember, findRoutine을 넣는 건 별로일까요?
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.
논의 완료!
public ResponseEntity<Response> createMemberDailyRoutine( | ||
Principal principal, @RequestBody MemberDailyRoutineRequest request) { | ||
val memberId = Long.parseLong(principal.getName()); | ||
val response = memberDailyRoutineService.createMemberDailyRoutine(memberId, request); |
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.
memberId에 Long.parseLong(principal.getName())을 넣는 건 별로일려나요??
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.
논의 완료!
✨ Related Issue
📝 기능 구현 명세
🐥 추가적인 언급 사항