Skip to content

[Fix] Q&A 목록 페이지 자동 해결 처리 제거 및 댓글 중복 버그 수정#227

Merged
kkw610 merged 4 commits into
developfrom
fix/#226
Jun 19, 2026
Merged

[Fix] Q&A 목록 페이지 자동 해결 처리 제거 및 댓글 중복 버그 수정#227
kkw610 merged 4 commits into
developfrom
fix/#226

Conversation

@kkw610

@kkw610 kkw610 commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

#️⃣연관된 이슈

#226

📝작업 내용

1. 운영진 댓글 작성 시 자동 '해결됨' 처리 제거

  • handleCommentSubmit에서 isStaff일 때 PATCH /status를 호출하던 코드 제거
  • 해결 처리는 QnADetailPage.js의 '해결됨으로' 버튼(handleResolve)을 통한
    수동 조작으로만 가능하도록 통일
  • 기존 동작(해결된 질문에 댓글이 달리면 미해결로 전환)은
    백엔드 QuestionService.createComment에 이미 구현되어 있어 변경 없음

2. 해결된 질문 카드 옅은 회색 배경 처리

  • questionCard에 question.isResolved 여부에 따라
    questionCardResolved 클래스를 조건부로 추가
  • CSS에서 --gray50(#F5F5F5) 배경 적용
  • 정렬상 아래로 내려가는 기존 동작과 함께 시각적으로도
    미해결 질문과 쉽게 구분 가능

3. 댓글 중복 등록 버그 수정

  • handleCommentSubmit에서 API 응답 직후 직접 previewComments를
    갱신하던 낙관적 업데이트 코드 제거
  • SSE comment-created 이벤트(handleCommentCreatedEvent) 단일 경로로만
    댓글 미리보기 갱신하도록 통일

수정 파일

  • frontend/src/pages/qna/QnAListPage.js
  • frontend/src/pages/qna/QnAListPage.module.css

영향받지 않는 부분

  • frontend/src/pages/qna/QnADetailPage.js (이미 올바르게 구현되어 있어 수정 안 함)
  • backend QuestionService.java (자동 해결 로직이 없었음을 확인, 수정 불필요)

🛠️테스트 방법

  1. 운영진 계정으로 로그인 → Q&A 목록에서 미해결 질문에 댓글 작성
    → 질문이 자동으로 해결됨 처리되지 않는지 확인
  2. 동일 질문에 다시 댓글 작성 → 미리보기에 댓글이 1개만 추가되는지 확인
  3. 상세페이지에서 '해결됨으로' 버튼으로 수동 해결 처리
    → 목록으로 돌아왔을 때 해당 질문 카드가 옅은 회색으로 표시되는지 확인
  4. 해결된 질문에 일반 사용자가 댓글 작성
    → 미해결로 전환되고 카드 색상도 흰색으로 돌아오는지 확인

Closes #226

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
piro-in Ready Ready Preview, Comment Jun 19, 2026 5:50am
piro-in-765d Ready Ready Preview, Comment Jun 19, 2026 5:50am

@kkw610 kkw610 merged commit b1bf9c9 into develop Jun 19, 2026
4 checks passed
@kkw610 kkw610 deleted the fix/#226 branch June 19, 2026 06:08
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.

[Bug] Q&A 목록 페이지 운영진 댓글 자동 해결 처리 / 해결 질문 미구분 / 댓글 중복 등록 수정

1 participant