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

refactor : improve follow api #311

Merged
merged 6 commits into from
Jan 6, 2025
Merged

refactor : improve follow api #311

merged 6 commits into from
Jan 6, 2025

Conversation

chadongmin
Copy link
Contributor

@chadongmin chadongmin commented Jan 3, 2025

Resolves #{이슈-번호}

해결하려는 문제가 무엇인가요?

  • 특정 유저가 팔로잉하고 있는 유저 정보

  • 특정 유저를 팔로우하고 있는 유저 정보

  • 위 두 가지 케이스를 각각 api로 분리했습니다.

  • follow 엔티티의 User 타입으로 가지고 있던 follower_user_id 필드를 Long으로 수정했습니다.

어떻게 해결했나요?

어떤 부분에 집중하여 리뷰해야 할까요?

참고 자료


RCA 룰

r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

@chadongmin chadongmin requested a review from Whale0928 January 3, 2025 14:38
@chadongmin chadongmin self-assigned this Jan 3, 2025
@chadongmin chadongmin linked an issue Jan 3, 2025 that may be closed by this pull request
@chadongmin chadongmin marked this pull request as draft January 3, 2025 14:38
@chadongmin chadongmin marked this pull request as draft January 3, 2025 14:38
@chadongmin chadongmin marked this pull request as ready for review January 3, 2025 14:58
Copy link
Collaborator

@Whale0928 Whale0928 left a comment

Choose a reason for hiding this comment

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

이전 리뷰때는 컨펌했지만, 수정사항을 보면서 이 구조에 대해 다시 생각해보게 되네요.

타입 파라미터로 두 API를 합친게 어떤 장점이 있을지 궁금합니다. 응답도 다르고 책임도 모호해진 것 같은데, 확장성이 고려하신건가요? 그렇다면 새로운 타입 추가마다 if문 분기가 늘어날 텐데 이 방향이 맞을지 고민이 되네요.

/following, /followers로 나뉜 기존 구조도 충분히 직관적이었던 것 같은데, 어떻게 생각하시나요?

@chadongmin
Copy link
Contributor Author

이전 리뷰때는 컨펌했지만, 수정사항을 보면서 이 구조에 대해 다시 생각해보게 되네요.

타입 파라미터로 두 API를 합친게 어떤 장점이 있을지 궁금합니다. 응답도 다르고 책임도 모호해진 것 같은데, 확장성이 고려하신건가요? 그렇다면 새로운 타입 추가마다 if문 분기가 늘어날 텐데 이 방향이 맞을지 고민이 되네요.

/following, /followers로 나뉜 기존 구조도 충분히 직관적이었던 것 같은데, 어떻게 생각하시나요?

리뷰 반영하여 수정했습니다 리뷰 부탁드려요 👍

@chadongmin chadongmin requested a review from Whale0928 January 5, 2025 14:34
Copy link
Collaborator

@Whale0928 Whale0928 left a comment

Choose a reason for hiding this comment

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

필드명과 생성방식만 수정하고 마무리하죠

@chadongmin chadongmin force-pushed the pr/309-improve-follow-api branch from 0db20c7 to e9bef03 Compare January 6, 2025 12:55
Copy link

sonarqubecloud bot commented Jan 6, 2025

@chadongmin chadongmin requested a review from Whale0928 January 6, 2025 12:58
@Whale0928 Whale0928 merged commit e8ba148 into dev Jan 6, 2025
3 checks passed
@Whale0928 Whale0928 deleted the pr/309-improve-follow-api branch January 6, 2025 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

improve follow api
2 participants