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.

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

[SAMBAD-267] 릴레이 질문 hidden 저장 및 조회 필터링 기능 추가 #117

Merged
merged 2 commits into from
Aug 20, 2024

Conversation

nahyeon99
Copy link
Member

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

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

📝 개요

  • 자기소개 페이지 내, 릴레이 질문 답변의 숨김 처리 요청 api 를 추가합니다.
  • 본인의 작성한 릴레이 질문 목록 조회 시, isHidden 응답 필드를 추가합니다. (기존 api 수정)
  • 다른 멤버의 릴레이 질문 목록 조회 시, isHidden 으로 필터링 후 반환합니다. (기존 api 수정)

‼️ 주의사항

  • MeetingAnswer에 isHidden 필드가 required로 추가되었습니다. 쿼리 업데이트가 필요합니다.

@nahyeon99 nahyeon99 requested review from kkjsw17 and LeeHanEum August 20, 2024 16:10
@nahyeon99 nahyeon99 self-assigned this Aug 20, 2024
@kkjsw17 kkjsw17 force-pushed the feature/SAMBAD-267-post_meeting_question_hidden branch from 69034da to 9af70b0 Compare August 20, 2024 16:19
Copy link

Unit Test Results

4 tests   4 ✔️  0s ⏱️
1 suites  0 💤
1 files    0

Results for commit 9af70b0.

Copy link
Collaborator

@kkjsw17 kkjsw17 left a comment

Choose a reason for hiding this comment

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

LGTM

@nahyeon99 nahyeon99 merged commit a9a9c13 into develop Aug 20, 2024
2 checks passed
@nahyeon99 nahyeon99 deleted the feature/SAMBAD-267-post_meeting_question_hidden branch August 20, 2024 16:29
kkjsw17 pushed a commit that referenced this pull request Aug 23, 2024
* feat: 모임 질문 숨김 요청 기능 추가

* feat: 작성한 모임 질문 답변 히스토리 조회 시 hidden 처리 추가
nahyeon99 added a commit that referenced this pull request Aug 24, 2024
* feat: 모임 질문 숨김 요청 기능 추가

* feat: 작성한 모임 질문 답변 히스토리 조회 시 hidden 처리 추가
nahyeon99 added a commit that referenced this pull request Aug 24, 2024
* feat: 모임 질문 숨김 요청 기능 추가

* feat: 작성한 모임 질문 답변 히스토리 조회 시 hidden 처리 추가
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.

2 participants