-
Notifications
You must be signed in to change notification settings - Fork 6
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
Auth, Notification 도메인 리팩토링 #538
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.
테바짱굿~ 깔끔하네요 👍👍👍👍
public List<String> findAllByRecipientId(long recipientId) { | ||
return fcmTokenRepository.findAllTokenByMemberId(recipientId); | ||
} | ||
|
||
public List<String> findAllByRecipients(List<Long> recipients) { | ||
return fcmTokenRepository.findAllTokenByMemberIds(recipients); | ||
} |
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.
이 부분은 일급컬렉션으로 만들지 않아도 될까요??
memberNotifications | ||
.stream() | ||
.map(MemberNotification::getMoudaNotification) | ||
.map(NotificationFindAllResponse::from) |
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.
얘는 왜 from 이져/??
Member newMember = Member.builder() | ||
.kakaoId(kakaoId) | ||
.build(); | ||
memberWriter.append(newMember); |
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.
save
대신에 append
라는 메서드명을 사용한 이유가 뭘까요?
|
||
private final KakaoOauthClient kakaoOauthClient; | ||
|
||
// TODO : 단위 테스트 작성 -> 외부 API는 어떻게 테스트하는 것이 좋은가? 해야하는가? |
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.
테바오,,테바오,,,오바테,,테오바,,,오바,,
PR의 목적이 무엇인가요?
Auth, Notification 서비스의 복잡성을 개선하기 위해 Implement layer 를 도입하여 리팩토링
이슈 ID는 무엇인가요?
설명
질문 혹은 공유 사항 (Optional)