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.

Sign up for GitHub

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

Jump to bottom

[FEAT] 답변 조회 API 변경사항 수정 #138

Merged
merged 16 commits into from
Sep 10, 2024
Merged

[FEAT] 답변 조회 API 변경사항 수정 #138

merged 16 commits into from
Sep 10, 2024

Conversation

kyxxgsoo
Copy link
Contributor

@kyxxgsoo kyxxgsoo commented Sep 3, 2024 •
edited
Loading

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

feat/#137/answerAPI -> develop

변경 사항

  • 답변 조회 API Response DTO 변경 & 해당 답변에 좋아요 눌렀는지 여부 조사 함수 구현
  • 특정 게시글에 좋아요 눌렀는지 여부 조사 함수 구현
  • 게시글 댓글 리스트 조회 API Response DTO 변경
  • 게시글 좋아요/취소 API 오류 수정
  • 질문 commentCount 관련 로직 구현
  • 질문 좋아요/취소 API 구현 및 질문 조회 API 적용
  • AnswerCommentInfo DTO 변경

@kyxxgsoo kyxxgsoo linked an issue Sep 3, 2024 that may be closed by this pull request
[FEAT] 답변 조회 API 변경사항 수정 #137
Closed
2 tasks
@kyxxgsoo kyxxgsoo force-pushed the feat/#137/answerAPI branch 3 times, most recently from a3db6a5 to fe0e850 Compare September 3, 2024 19:26
tnals2384
tnals2384 previously approved these changes Sep 4, 2024
View reviewed changes
Copy link
Contributor

@tnals2384 tnals2384 left a comment

Choose a reason for hiding this comment

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

LGTM

youngeun-dev
youngeun-dev previously approved these changes Sep 4, 2024
View reviewed changes
Copy link
Contributor

@youngeun-dev youngeun-dev left a comment

Choose a reason for hiding this comment

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

LGTM !!!

@kyxxgsoo kyxxgsoo dismissed stale reviews from youngeun-dev and tnals2384 via b1dd220 September 8, 2024 07:50
@kyxxgsoo kyxxgsoo added the ✨feature New feature or requeest label Sep 9, 2024
tnals2384
tnals2384 approved these changes Sep 9, 2024
View reviewed changes
Copy link
Contributor

@tnals2384 tnals2384 left a comment

Choose a reason for hiding this comment

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

LGTM

jaewonLeeKOR
jaewonLeeKOR approved these changes Sep 9, 2024
View reviewed changes
Copy link
Member

@jaewonLeeKOR jaewonLeeKOR left a comment

Choose a reason for hiding this comment

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

LGTM

@tnals2384 tnals2384 merged commit 723297e into develop Sep 10, 2024
1 check passed
@tnals2384 tnals2384 mentioned this pull request Sep 12, 2024
Merged
4 tasks
@jaewonLeeKOR jaewonLeeKOR deleted the feat/#137/answerAPI branch September 25, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@youngeun-dev youngeun-dev youngeun-dev left review comments

@jaewonLeeKOR jaewonLeeKOR jaewonLeeKOR approved these changes

@tnals2384 tnals2384 tnals2384 approved these changes

Assignees
No one assigned
Labels
✨feature New feature or requeest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] 답변 조회 API 변경사항 수정
4 participants
@kyxxgsoo @jaewonLeeKOR @youngeun-dev @tnals2384