Skip to content

[#451][FIX] canWritePreOpinions 조건 변경 - #452

Merged
JWoong-01 merged 1 commit into
devfrom
fix/confrim-topics
Jun 17, 2026
Merged

[#451][FIX] canWritePreOpinions 조건 변경#452
JWoong-01 merged 1 commit into
devfrom
fix/confrim-topics

Conversation

@JWoong-01

Copy link
Copy Markdown
Contributor

PR 요약

이 PR이 어떤 변경을 하는지 간단히 설명하고, 체크 표시는 괄호 사이에 소문자 'x'를 삽입하세요.

  • 기능 추가
  • 버그 수정
  • 코드 리팩토링
  • 문서 수정
  • 기타 (설명)

확정된 주제 조회 API의 사전의견 액션 값이 약속 참가 여부를 반영하도록 수정합니다.


이슈 번호


주요 변경 사항

주요 파일, 로직, 컴포넌트 등을 구체적으로 적어주세요.

  • TopicService: 확정된 주제 조회 시 약속 참가 여부를 확인해 사전의견 작성/조회 액션을 계산하도록 변경
  • TopicServiceTest: 약속 참가자가 아닌 경우 canWritePreOpinionsfalse로 내려가는 테스트 추가

참고 사항

리뷰어가 알아야 할 추가 정보, 테스트 방법 등을 작성해주세요.

  • 관련 API 엔드포인트: GET /api/gatherings/{gatheringId}/meetings/{meetingId}/confirm-topics
  • 영향 응답 필드:
    • actions.canViewPreOpinions
    • actions.canWritePreOpinions
  • 로컬 테스트 방법:
    • ./gradlew test --tests com.dokdok.topic.service.TopicServiceTest

@JWoong-01 JWoong-01 self-assigned this Jun 17, 2026
@JWoong-01 JWoong-01 added the type:fix 버그 label Jun 17, 2026
@JWoong-01
JWoong-01 merged commit 72ec732 into dev Jun 17, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in dokdok-server Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:fix 버그

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant