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

[LIME-9] 친구 관계 기능 추가 #15

Merged
merged 20 commits into from
Jan 23, 2024
Merged

[LIME-9] 친구 관계 기능 추가 #15

merged 20 commits into from
Jan 23, 2024

Conversation

Yiseull
Copy link
Member

@Yiseull Yiseull commented Jan 21, 2024

📌 PR 종류

어떤 종류의 PR인지 아래 항목 중에 체크 해주세요.

  • 🐛 버그 수정
  • ✨ 기능 추가
  •  테스트 추가
  • 🎨 코드 스타일 변경 (formatting, local variables)
  • 🔨 리팩토링 (기능 변경 X)
  • 💚 빌드 관련 수정
  • 📝 문서 내용 수정
  • 그 외, 어떤 종류인지 기입 바람:

📌 어떤 기능이 추가 되었나요?

Issue Number

LIME-9

🩵 기능 설명

팔로우 기능

주소: /follow/{nickname}

- {nickname}을 가진 회원을 팔로우한다.
- 이미 팔로우가 되어 있으면 예외가 터진다. 

언팔로우 기능

주소: /unfollow/{nickname}

- {nickname}을 가진 회원을 언팔로우한다.
- 팔로우가 되어 있지 않으면 예외가 터진다. 

팔로워 목록 조회

주소: /follower/{nickname}

- {nickname}을 가진 회원의 팔로워 목록을 커서 페이지네이션 방식으로 조회한다.

팔로잉 목록 조회

주소: /following/{nickname}

- {nickname}을 가진 회원의 팔로잉 목록을 커서 페이지네이션 방식으로 조회한다.

📌 기존에 있던 기능에 영향을 주나요?

  • 아니요
  • 마이페이지 조회할 때 해당 회원의 팔로워 수팔로잉 수도 조회되도록 추가하였습니다.

@Yiseull Yiseull added enhancement New feature or request test This will not be worked on labels Jan 21, 2024
@Yiseull Yiseull self-assigned this Jan 21, 2024
Copy link
Member

@HandmadeCloud HandmadeCloud left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

Comment on lines 61 to 62
@Operation(summary = "팔로잉 목록 조회", description = "회원이 팔로우한 사람들의 목록을 조회한다.")
@GetMapping("/following/{nickname}")
Copy link
Member

Choose a reason for hiding this comment

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

설명 팔로잉으로 고치면 될거같아욤

Copy link
Member Author

@Yiseull Yiseull Jan 23, 2024

Choose a reason for hiding this comment

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

6197520

설명 수정했습니다! 😄

Comment on lines +32 to +38
@ManyToOne(fetch = FetchType.LAZY)
@JoinColumn(name = "to_member_id", nullable = false)
private Member toMember;

@ManyToOne(fetch = FetchType.LAZY)
@JoinColumn(name = "from_member_id", nullable = false)
private Member fromMember;
Copy link
Member

Choose a reason for hiding this comment

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

궁금한게 manyToOne으로 연관관계를 맺어줬을 때 발생하는 트러블 슈팅은 없었나요? 맞팔로잉 상태면 순환 관련 문제가 있을 수 있지 않을까 예상했는데 단방향이라 없을 것 같기도 하네요

Copy link
Member Author

Choose a reason for hiding this comment

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

단방향이여서 순환참조 문제는 없을 것 같아요!! 생기면 바로 공유하겠습니다😎

Copy link
Contributor

@Curry4182 Curry4182 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 전체적으로 코드를 봤을 때 따로 컨맨트 할게 없어서 바로 승인하였습니다 😊

@Yiseull Yiseull merged commit ed24bee into main Jan 23, 2024
@Yiseull Yiseull deleted the LIME-9-add-follow branch January 23, 2024 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request test This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants