fix: 배너 '현재 노출'이 링크 대상 숨김 배너를 건너뛰고 '링크 대상 숨김' 표시 - #67
Merged
Merged
Conversation
- AdminBanner.linkTargetAvailable 추가분(BE feat/admin-banner-link-target e76ccf9 기준) - BE 머지 뒤 develop 기준으로 다시 동기화한다
연결한 상품·매장·카테고리가 나중에 숨겨지거나 삭제된 배너도 '현재 노출'로 표시했다. 구매자 앱은 그런 배너를 건너뛰고 다음 배너를 보여 주는데, 목록은 링크 대상 상태를 몰랐다. - AdminBanners·AdminBannersVisible 문서에 linkTargetAvailable 추가(BE가 구매자 조회와 같은 조건으로 판정) - currentBannerIds: linkTargetAvailable=false면 슬롯 후보에서 빼 구매자 앱과 같은 배너를 고름 - 목록 노출 상태 칸에 '링크 대상 숨김'(caution) 배지, 페이지 설명 문구 갱신 회귀 테스트 - live-status.spec: 대상 숨김인 상위 배너를 건너뛰고 다음 배너가 현재 노출 - banners.spec: 상위 배너는 '노출 중 + 링크 대상 숨김', 다음 배너가 '현재 노출' - 반증: 건너뛰기를 되돌리면 2건, 배지를 빼면 1건 실패
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This was referenced Oct 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
배경
linkTargetAvailable(구매자 앱과 같은 기준)을 내려 주게 됐습니다(BE PR 반영 후 스키마 동기화).변경
currentBannerIds)이linkTargetAvailable이 false인 배너를 건너뜁니다. 구매자 앱과 같은 순서입니다.StatusPill(caution)입니다.linkTargetAvailable을 추가했습니다.테스트
live-status.spec: 최우선 배너의 대상이 숨겨지면 다음 배너를 현재 노출로 고르는지 확인했습니다.banners.spec: 최우선 배너에는 '링크 대상 숨김'이, 다음 배너에는 '현재 노출'이 붙는지 확인했습니다.플랜 대조