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

[BE] refactor: reviewRequestCode를 받는 api를 reviewGroupId를 받도록 변경 #1183

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Kimprodp
Copy link
Contributor

@Kimprodp Kimprodp commented Mar 2, 2025


🚀 어떤 기능을 구현했나요 ?

🔥 어떻게 해결했나요 ?

  • 프론트에서 reviewGroupId를 사용할 수 있게, /v2/groups/summary의 응답으로 reviewGroupId를 포함하도록 했습니다.
  • 이후 reviewRequestCode로 받던 api를 모두 reviewGroupId로 변경했습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 이전에 네이밍이 groupId 였는데, 코드 내에서는 명칭을 통일하는 것이 좋을 것 같아 reviewGroupId로 했습니다. 혹시 groupId가 더 나으려나요?
  • 추가로 응답 시, reviewGroupId를 포함해야 하는 부분이 더 있을까요?

📚 참고 자료, 할 말

  • 백엔드 협의 후 프론트 안내가 필요합니다.

Copy link

github-actions bot commented Mar 2, 2025

Test Results

206 tests   203 ✅  6s ⏱️
 75 suites    3 💤
 75 files      0 ❌

Results for commit 0d42c7b.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[BE] URI에서 ReviewRequestCode로 받는 부분을 ReviewGroupId를 사용하도록 변경한다.
1 participant