-
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
pr Feat/9 member post comment mapping #53
Conversation
활성화 꿀팁공유해요 조회시 (findByIdActivated) Member를 fetchjoin하여 조회
메서드 직접 사용하도록 수정 및 import
자격 검증 및 이미 존재시에 생성하지 않고 return
좋아요, 댓글 수를 합하여 TOP5 추출, ToDo 스크랩 수 계산
기존 소셜로그인 - originName으로 변경, 관심사 Member(1) : Interest(N)로 매핑, 기존 값 수정 Editor 사용
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.
준형님 연휴에도 작업하느라 고생 많으셨습니다! 꼼꼼히 확인하고 리뷰 순차적으로 달아보겠습니다~~
src/main/java/com/api/ttoklip/domain/honeytip/post/service/HoneyTipCommonService.java
Outdated
Show resolved
Hide resolved
삭제할 때 content를 "식제된 댓글입니다." 라는 내용으로 보여주기 위해 댓글 deactivate 오버라이드, 단건 조회시 비활성화도 댓글 목록에 보이도록 수정
src/main/java/com/api/ttoklip/domain/honeytip/post/dto/response/HoneyTipSingleResponse.java
Outdated
Show resolved
Hide resolved
public interface MemberRepository extends JpaRepository<Member, Long> { | ||
Optional<Member> findByNaverEmail(String email); | ||
Optional<Member> findByKakaoId(Long kakaoId); | ||
boolean existsByNickname(String nickname); |
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.
CustomOAuth2UserService
코드에 findByNaverEmail 과 findByKakaoId 는 있는데
existsByNickname
는 어떻게 처리되는지 궁금합니다!! DB에 중복되는 nickname이 있는지 JPA가 확인해 주는 건가요?
연휴에 작업 수고하셨습니다~! |
fix web 로그인 방식 sdk 로그인 방식 활용
… into 54-fix-web-로그인-방식-sdk-로그인-방식-활용
54 fix web 로그인 방식 sdk 로그인 방식 활용
Issue number and Link
이슈 번호
Summary
PR Type
Other Information
DB가 좀 바뀌어, 배포 DB는 ddoklip-prod 스키마로, 개발 DB는 ddoklip-develop으로 나눴습니다.
TODO