Skip to content

Commit 5ecdd90

Browse files
authored
Merge pull request #120 from IT-Cotato/feature/118
refactor: 마이페이지 배너 매거진 기본값 제거 및 비활성 배너 반환 처리
2 parents 8df95f3 + 6c3bcc1 commit 5ecdd90

File tree

4 files changed

+9
-14
lines changed

4 files changed

+9
-14
lines changed

src/main/java/com/ongil/backend/domain/banner/controller/BannerController.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ public class BannerController {
2323
private final BannerService bannerService;
2424

2525
@GetMapping
26-
@Operation(summary = "배너 조회 API", description = "토큰 필요. 현재 사용자에게 보여줄 배너를 조회합니다. 구매 직후 리뷰 유도, 한달 후기 유도, 매거진 추천 중 우선순위에 따라 반환됩니다.")
26+
@Operation(summary = "배너 조회 API", description = "토큰 필요. 현재 사용자에게 보여줄 배너를 조회합니다. 구매 직후 리뷰 유도, 한달 후기 유도 우선순위에 따라 반환되며, 해당 배너가 없으면 enabled=false로 반환됩니다.")
2727
public ResponseEntity<DataResponse<BannerResponse>> getBanner(
2828
@AuthenticationPrincipal Long userId
2929
) {

src/main/java/com/ongil/backend/domain/banner/converter/BannerConverter.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,4 +19,10 @@ public BannerResponse toResponse(BannerType type, String title, String buttonTex
1919
.enabled(enabled)
2020
.build();
2121
}
22+
23+
public BannerResponse toEmptyResponse() {
24+
return BannerResponse.builder()
25+
.enabled(false)
26+
.build();
27+
}
2228
}

src/main/java/com/ongil/backend/domain/banner/enums/BannerType.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
@Getter
77
@RequiredArgsConstructor
88
public enum BannerType {
9-
MAGAZINE("매거진 유도"),
109
REVIEW_PROMPT("구매 직후 리뷰 작성 유도"),
1110
MONTHLY_REVIEW_PROMPT("한달 후 리뷰 작성 유도");
1211

src/main/java/com/ongil/backend/domain/banner/service/BannerService.java

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@ public BannerResponse getBanner(Long userId) {
4444
return monthlyReviewBanner;
4545
}
4646

47-
// 3순위: 매거진 유도 알림
48-
return createMagazineBanner();
47+
// 리뷰 배너가 없으면 비활성 배너 반환
48+
return bannerConverter.toEmptyResponse();
4949
}
5050

5151
private BannerResponse checkInitialReviewBanner(Long userId) {
@@ -141,14 +141,4 @@ private BannerResponse checkMonthlyReviewBanner(Long userId) {
141141
return null;
142142
}
143143

144-
private BannerResponse createMagazineBanner() {
145-
return bannerConverter.toResponse(
146-
BannerType.MAGAZINE,
147-
"추천 매거진을 확인해보세요",
148-
"보러가기",
149-
"/magazine",
150-
null,
151-
true
152-
);
153-
}
154144
}

0 commit comments

Comments
 (0)