-
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] 회원 데일리 루틴 달성 #39
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.
이제는 더 할 말이 없을정도... ㅜㅜ 너무 고생하셨습니다
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.
대단합니다..
@@ -1,8 +1,11 @@ | |||
package com.soptie.server.memberRoutine.service; | |||
|
|||
import com.soptie.server.memberRoutine.dto.AchievedMemberDailyRoutineResponse; | |||
import com.soptie.server.memberRoutine.dto.MemberDailyRoutineRequest; | |||
import com.soptie.server.memberRoutine.dto.MemberDailyRoutineResponse; | |||
|
|||
public interface MemberDailyRoutineService { | |||
MemberDailyRoutineResponse createMemberDailyRoutine(long memberId, MemberDailyRoutineRequest 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.
Long이 아니고 long인 이유가 있을까요??
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.
null을 허용하지 않는 인자이기 때문에 원시형을 사용했습니다!
|
||
@Override | ||
@Transactional | ||
public AchievedMemberDailyRoutineResponse achieveMemberDailyRoutine(long memberId, Long routineId) { |
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.
여기도 long인 이유가 궁금합니다.
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
📝 기능 구현 명세
🐥 추가적인 언급 사항