Skip to content

feat: 최신 develop 반영 및 GroupedOpenApi 설정 추가#33

Closed
sohyunxxi wants to merge 10 commits intodevelopfrom
feature/#17
Closed

feat: 최신 develop 반영 및 GroupedOpenApi 설정 추가#33
sohyunxxi wants to merge 10 commits intodevelopfrom
feature/#17

Conversation

@sohyunxxi
Copy link
Collaborator

@sohyunxxi sohyunxxi commented Mar 17, 2025

#️⃣ Issue Number

feature/#17

📝 요약(Summary)

새로운 API인 question & answer 파트를 추가함
그리고 merge 할 때 충돌 방지를 위해 다시 develop에서 pull을 한 후 다시 push 수행

🛠️ PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📸스크린샷 (선택)

image

💬 공유사항 to 리뷰어

question이랑 answer api를 현재 같이 그룹화하여 작성했는데, 진행 상황에 대한 조회 및 저장하는 api, 참여 인원 카운팅은 따로 api를 그룹화하는게 좋을까요? 아니면 이 questionAPI에 같이 하는게 좋을까요?

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@sohyunxxi sohyunxxi added the feature feature label Mar 17, 2025
@sohyunxxi sohyunxxi requested a review from Pochomo March 17, 2025 07:03
@sohyunxxi sohyunxxi self-assigned this Mar 17, 2025
@gangjune
Copy link
Collaborator

아직 api분류 관련해서는 자세히 논의된 바가 없는 것으로 알고 있는데, 제 생각에는 진행 상황에 대한 api는 질문 답변과 밀접하게 관련이 있는 것 같아 같이 그룹화해도 좋을 것 같고, 참여 인원 카운팅은 따로 하는 것도 괜찮을 것 같습니다!

@Pochomo
Copy link
Collaborator

Pochomo commented Mar 17, 2025

그룹화하는게 좋을거 같습니다. swaggerconfig 관련 수정사항은 카톡으로 한번 더 말해서 빠른 시일내에 정하는게 좋을거같습니다.

@sohyunxxi sohyunxxi closed this Mar 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants