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

feat: 친구 맺기 API 개발 #25

Merged
merged 6 commits into from
Jan 3, 2024
Merged

feat: 친구 맺기 API 개발 #25

merged 6 commits into from
Jan 3, 2024

Conversation

devxb
Copy link
Member

@devxb devxb commented Jan 2, 2024

🚀 어떤 기능을 개발했나요?

친구 맺기 기능을 추가했습니다.

🕶️ 어떻게 해결했나요?

  • 친구 맺기 통합테스트 작성
  • User 도메인에 친구 필드 추가
  • Login된 유저 정보 교환에 LoginContext 사용하도록 구성
  • 통합테스트에 테스트용 LoginContext 구성 추가
  • 유저 업데이트 API에서 Login된 유저 id를 사용해서 업데이트 하도록 수정

🦀 이슈 넘버

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

User 도메인에 친구필드를 id로 추가했는데, 현재 역정규화 하는거보다 id로 들고있는게 더 좋을거 같아서 이렇게 구성했습니다. 나중에 Friend에 대한 정보를 더 들고있어야할때 추가 하면 좋을거 같아요.

Copy link

sonarqubecloud bot commented Jan 3, 2024

@devxb devxb merged commit 246a88d into develop Jan 3, 2024
3 checks passed
@devxb devxb deleted the iss-#20 branch January 3, 2024 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 친구 신청 API
1 participant