Skip to content
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

Merged
merged 38 commits into from
Feb 12, 2024

Conversation

toychip
Copy link
Member

@toychip toychip commented Feb 10, 2024

Issue number and Link

이슈 번호

Summary

  1. Member와 게시글, 댓글 연관관계 매핑 & 좋아요 기능
  2. HoneyTip 수정 & 삭제시 작성자가 맞는지 검증 로직
  3. 댓글 삭제시 작성자가 맞는지 검증 로직 - CommonService에서 공통처리, 다른 Comment에서 수정 필요 x
  4. 회원가입 직후 프로필 사진, 관심사, 닉네임 삽입 로직
  5. 독립 경력 자동 증가 로직 - 스케줄러 사용
  6. HoneyTip & Question Main TOP5 조회 추가
  7. Oauth 로그인 책임을 백엔드에서 관리하는 웹 로그인 방식에서 로그인 책임을 안드로이드에서 관리하는 SDK로그인 방식으로 수정하였습니다.
  8. 안드로이드에서 로그인을 진행한 후 정보제공자로부터 엑세스 토큰과 provider를 서버에 주는 별도의 api를 만들어 자체적인 jwt token을 내려주도록 수정했습니다.

PR Type

  • Feature
  • Bugfix
  • Refactoring
  • Documentation
  • Other

Other Information

DB가 좀 바뀌어, 배포 DB는 ddoklip-prod 스키마로, 개발 DB는 ddoklip-develop으로 나눴습니다.

TODO

  1. 게시글 성격 Entity 수정 삭제 검증로직 필요 (Question, HoneyTip는 완료)
  2. 게시글 성격 단건 조회, 페이징 조회시 Response Dto에 작성자 필드 추가 (Question, HoneyTip는 완료)
  3. 상속받은 자식 댓글 생성 로직에서 member 부분 getCurrentmember로 생성후 삽입 - 확인 부탁드려요 ❗️

활성화 꿀팁공유해요 조회시 (findByIdActivated) Member를 fetchjoin하여 조회
메서드 직접 사용하도록 수정 및 import
자격 검증 및 이미 존재시에 생성하지 않고 return
좋아요, 댓글 수를 합하여 TOP5 추출, ToDo 스크랩 수 계산
기존 소셜로그인 - originName으로 변경, 관심사 Member(1) : Interest(N)로 매핑, 기존 값 수정 Editor 사용
@toychip toychip added ✨ Feature 새로운 기능 추가 🔀 PullRequest Branch 합병 임준형 labels Feb 10, 2024
@toychip toychip self-assigned this Feb 10, 2024
@toychip toychip linked an issue Feb 10, 2024 that may be closed by this pull request
12 tasks
Copy link
Contributor

@why-only-english why-only-english left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

준형님 연휴에도 작업하느라 고생 많으셨습니다! 꼼꼼히 확인하고 리뷰 순차적으로 달아보겠습니다~~

삭제할 때 content를 "식제된 댓글입니다." 라는 내용으로 보여주기 위해 댓글 deactivate 오버라이드, 단건 조회시 비활성화도 댓글 목록에 보이도록 수정
Comment on lines 7 to 10
public interface MemberRepository extends JpaRepository<Member, Long> {
Optional<Member> findByNaverEmail(String email);
Optional<Member> findByKakaoId(Long kakaoId);
boolean existsByNickname(String nickname);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CustomOAuth2UserService 코드에 findByNaverEmailfindByKakaoId 는 있는데
existsByNickname 는 어떻게 처리되는지 궁금합니다!! DB에 중복되는 nickname이 있는지 JPA가 확인해 주는 건가요?

@psae0714
Copy link
Contributor

연휴에 작업 수고하셨습니다~!

@toychip toychip merged commit 0459010 into develop Feb 12, 2024
@toychip toychip deleted the Feat/9-Member-Post-Comment-Mapping branch October 17, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 새로운 기능 추가 🔀 PullRequest Branch 합병 임준형
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Member와 게시글 댓글 연관관계 매핑 & 좋아요 기능
3 participants