[Fix/#315] 멤버의 구독 정보 제공을 위한 조회시 취소를 구분하지 않은 문제 해결 #316
Conversation
| .leftJoin(MappingWorkbookArticle.MAPPING_WORKBOOK_ARTICLE) | ||
| .join(MappingWorkbookArticle.MAPPING_WORKBOOK_ARTICLE) | ||
| .on(SUBSCRIPTION.TARGET_WORKBOOK_ID.eq(MappingWorkbookArticle.MAPPING_WORKBOOK_ARTICLE.WORKBOOK_ID)) | ||
| .where(SUBSCRIPTION.MEMBER_ID.eq(query.memberId)) | ||
| .and(SUBSCRIPTION.TARGET_MEMBER_ID.isNull) | ||
| .and(SUBSCRIPTION.UNSUBS_OPINION.`in`(query.unsubOpinion, query.activeSubscriptionUnsubOpinion)) |
There was a problem hiding this comment.
기존 쿼리의 경우 leftJoin + 구독 취소 사유 관련 조건 x 로 멤버의 구독 정보를 모두 조회하고 있었습니다.
leftJoin -> join 변경 + 구독 취소 사유 조건 추가 "recieve.all"(구독 완료), "" (구독 중) 조건을 추가하여 구독 완료, 구독 중인 구독 정보만 조회하도록 수정하였습니다.
There was a problem hiding this comment.
UNSUBS_OPINION 이게 유저가 입력하는 텍스트인데 저렇게 비교가 가능한가요?
There was a problem hiding this comment.
취소할 때 유저가 입력하는데
저희가 입력하는 값도 존재하거든요!
"recieve.all" 의 경우 메일을 보내면서 학습지 끝까지 아티클을 보냈으면 저희가 기록하는 값이고
"" 의 경우는 구독을 하는 경우 기본 값이에요
취소하는 경우에 저런 경우가 안들어오길 바라는..
댓글 달면서 생각난 것은 취소하는 경우 ""처럼 아무것도 적지 않는 경우가 있다면 "cancel"과 같이 별도 처리 해줘야 하겠네요
There was a problem hiding this comment.
댓글 달면서 생각난 것은 취소하는 경우 ""처럼 아무것도 적지 않는 경우가 있다면 "cancel"과 같이 별도 처리 해줘야 하겠네요
f779f9d 에서 관련된 처리했습니다!
| @Valid @RequestBody | ||
| body: SubscribeWorkbookRequest, |
There was a problem hiding this comment.
토큰 사용으로 사용하지 않을 바디 삭제
* [Refactor/#275]: 인덱스 제약 수정 및 추가 (#294) * [Refactor/#300] 컨트롤러 인증 구현 (#301) * [Feat/#293] 작가 description json 필드 추가 - imageUrl(required) (#299) * [#298/refactor]: 가입/인증에 따라 다른 내용 채워서 이메일 보내도록 수정 (#302) * [Chore/#303] 단순 이메일 스타일 변경입니다 (#304) * [Refactor/#305] 액션 개선 및 수정 (#306) * [Fix/#308]: 시큐리티 필터 파싱 오류 해결 (#309) * [Feat/#310] 이미지 Webp 변환 코드 구현 (#311) * [Fix/#315] 멤버의 구독 정보 제공을 위한 조회시 취소를 구분하지 않은 문제 해결 (#316) * [#317/Refactor] 시큐리티 필터 제외 목록 점검 및 수정 (#318) --------- Co-authored-by: 지훈 <[email protected]> Co-authored-by: belljun3395 <[email protected]>
🎫 연관 이슈
resolved: #315
💁♂️ PR 내용
🙏 작업
🙈 PR 참고 사항
📸 스크린샷
데이터 : ID - 순서대로 2, 3, 4 상태 - 구독 완료, 구독 중, 취소
/api/v1/subscriptions/workbooks
에서 취소된 워크북을 제외하고 조회된 모습
/api/v1/subscriptions/workbooks/main
에서 취소된 워크북을 제외하고 조회된 모습
🤖 테스트 체크리스트