Skip to content

Conversation

@BH13KDR
Copy link
Collaborator

@BH13KDR BH13KDR commented Dec 12, 2025

✅ PR 요약

  • 작업 요약: 요청 받은 각종 피드백을 적용했습니다.

📄 상세 내용

  • 주요 변경 사항 1 : 각종 N+1문제를 select_related, prefetch_related 등을 적용해 해결했습니다.
  • 주요 변경 사항 2 : 각각 전달하던 데이터를 request.data로 일괄 전달하게 바꾸었습니다.
  • 주요 변경 사항 3 : 중복되던 코드를 리팩토링하고, 불필요한 코드를 제거했습니다.
  • 주요 변경 사항 4 : 유효성 검증 방식을 통일 했습니다.
  • 주요 변경 사항 5 : 에러 케이스 테스트 코드도 추가하였습니다.
  • 주요 변경 사항 6 : api명세서에 따른 response코드를 추가했습니다.
  • 주요 변경 사항 7 : tag를 schedule 에서 studygroup으로 변경했습니다.
  • 주요 변경 사항 8 : 누락된 created at, updated at을 추가했습니다.
  • 주요 변경 사항 9 : studygroup 파트가 커버리지 TEST를 통과하지 못하고 있습니다.
  • 주요 변경 사항 10 : update에서 누락된 created_at, updated_at을 수정했습니다

wjdtjddns98
wjdtjddns98 previously approved these changes Dec 15, 2025
@BH13KDR BH13KDR force-pushed the feat/schedule_PR branch 8 times, most recently from 7ea4787 to 1d62bf3 Compare December 18, 2025 01:14
BH13KDR added 13 commits December 18, 2025 11:03
#	apps/study_groups/urls.py

# Conflicts:
#	apps/study_groups/tests/tests.py
#	apps/study_groups/urls.py

# Conflicts:
#	apps/study_groups/urls.py
# ♻️ refactor : 리팩터링 : 각종 N+1 문제를 해결했습니다. 불필요한 코드를 제거했습니다.request.data로 data를 일괄 전달하게 바꾸었습니다. 중복되던 코드를 리팩토링했습니다. 유효성 검증 방식을 통일했습니다.
…했습니다. 변경사항에 맞는 결과 값을 검증하도록 test를 수정했습니다.
@SWWS97 SWWS97 merged commit 5587921 into develop Dec 18, 2025
2 checks passed
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.

3 participants