Skip to content

[Feat] 댓글 응답 및 SSE 이벤트에 최신 질문 상태 반영#133

Merged
issuejong merged 4 commits into
developfrom
fix/#132
May 29, 2026
Merged

[Feat] 댓글 응답 및 SSE 이벤트에 최신 질문 상태 반영#133
issuejong merged 4 commits into
developfrom
fix/#132

Conversation

@issuejong

Copy link
Copy Markdown
Collaborator

#️⃣연관된 이슈

#132

📝작업 내용

댓글 작성/조회 응답과 SSE 댓글 생성 이벤트에 최신 질문 상태 및 댓글 소유 여부 정보를 추가했습니다.

  • 댓글 작성 응답에 최신 질문 상태 추가

    • CommentCreateResisResolved 필드 추가
    • 댓글 작성 후 서버 내부 규칙까지 반영된 최신 질문 해결 상태 반환
    • 해결된 질문에 댓글이 달리면 미해결 상태로 자동 전환된 결과를 응답에 포함
  • 댓글 생성 SSE 이벤트에 최신 질문 상태 추가

    • CommentCreatedEventisResolved 필드 추가
    • 댓글 생성 이벤트 수신 시 프론트에서 질문 목록의 해결/미해결 상태를 함께 갱신할 수 있도록 처리
  • 댓글 조회 응답에 내 댓글 여부 추가

    • CommentResponseisMine 필드 추가
    • 현재 로그인 사용자가 작성한 댓글인지 여부를 반환
    • 최상위 댓글과 대댓글 모두 isMine 값을 포함하도록 처리
  • 질문 상태 변경 흐름 주석 보완

    • 관리자가 직접 질문 상태를 완료 처리하는 수동 전환과
    • 댓글 작성으로 인해 미해결로 자동 전환되는 서버 내부 도메인 규칙을 구분하여 주석 추가

@issuejong issuejong requested a review from Copilot May 29, 2026 14:17
@issuejong issuejong linked an issue May 29, 2026 that may be closed by this pull request
@issuejong issuejong self-assigned this May 29, 2026
@issuejong issuejong added the bug Something isn't working label May 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates comment-related DTOs and service wiring so REST and SSE responses include the latest question resolution state and comment ownership metadata.

Changes:

  • Adds isResolved to comment creation and comment-created SSE responses.
  • Adds isMine to comment detail responses for top-level comments and replies.
  • Clarifies comments around manual admin status changes versus automatic unresolved transitions.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
backend/src/main/java/com/example/Piroin/project/domain/question/service/QuestionService.java Wires new response fields and returns updated resolution state after comment creation.
backend/src/main/java/com/example/Piroin/project/domain/question/dto/QuestionResDTO.java Extends comment and SSE response records with new fields.
backend/src/main/java/com/example/Piroin/project/domain/question/controller/QuestionController.java Adds clarification comment for status update flow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@issuejong issuejong merged commit c641aaa into develop May 29, 2026
2 checks passed
@issuejong issuejong deleted the fix/#132 branch May 29, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] 댓글 작성 시 미해결 질문으로 미변경 오류

2 participants