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

[PC-627] authenticationService 의존성 제거 #66

Merged
merged 8 commits into from
Feb 18, 2025

Conversation

devchlee12
Copy link
Member

@devchlee12 devchlee12 commented Feb 16, 2025

🔗 관련 이슈

PC-627

✨ 작업 내용

  • matchedUserId 정보 추가
  • authenticationService 의존성 제거

✅ 체크리스트

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

🎃 새롭게 알게된 사항

📋 참고 사항

  • AuthenticationService에 대한 남은 의존성을 모두 제거했습니다.

@devchlee12 devchlee12 requested a review from Lujaec February 16, 2025 15:26
@devchlee12 devchlee12 self-assigned this Feb 16, 2025
Copy link

github-actions bot commented Feb 16, 2025

Unit Test Results

23 tests   22 ✔️  2s ⏱️
10 suites    0 💤
10 files      1

For more details on these failures, see this check.

Results for commit 5790ad9.

♻️ This comment has been updated with latest results.

Copy link
Member

@Lujaec Lujaec left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

그런데 이전에는 matchedUserId가 없어도 정상적으로 동작한 것 같던데 추가하신 이유가 있으실까요 ?

그리고 해당 PR은 클라이언트분들한테 머지해도 영향이 없는지 먼저 여쭤보고 머지되어야 될 것 같아요. 응답 형태가 변하기 앱 배포 심사가 진행중이라면 문제가 생길 수 있으니까요.

@devchlee12
Copy link
Member Author

추가한 이유는 IOS파트에서 각 API에서 모두 matchedUserId를 제공하달라는 요청이 있었기 때문입니다.
뷰모델이 나뉘어 있어서 각 API에서 제공해주지 않으면 비효율적인 API 접근을 해야만한다고 해서요!
디스코드 서버 채널에 차단 스레드 보시면 알 수 있으실거에요!

@devchlee12 devchlee12 merged commit d516d97 into develope Feb 18, 2025
0 of 2 checks passed
@devchlee12 devchlee12 changed the title [PC-627] Match 관련 조회 API들에 matchedUserId 모두 추가 [PC-627] authenticationService 의존성 제거 Feb 18, 2025
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.

2 participants