fix: 홈 팝업 노출 로직 수정

[seong-hui]

🤫 쉿, 나한테만 말해줘요. 이슈넘버

• close fix: homepopup 안뜨는 오류 수정 #2179

🧐 어떤 것을 변경했어요~?

🤔 그렇다면, 어떻게 구현했어요~?

❤️‍🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.

📸 스크린샷, 없으면 이것 참,, 섭섭한데요?

[coderabbitai]

Caution

Review failed

The pull request is closed.

Summary by CodeRabbit

릴리스 노트

• Bug Fixes
  • HomePopup의 표시 조건을 개선하여 특정 시나리오에서 더 적절하게 작동하도록 수정했습니다.

Walkthrough

HomePopupContainer.tsx에서 isLastGeneration 체크를 제거한 가드문이 삭제되었습니다. 이제 period와 showOnlyToRecentGeneration 조건을 만족하면, 세대 여부와 관계없이 HomePopup이 렌더링될 수 있습니다.

Changes

Cohort / File(s)	Summary
HomePopup 렌더링 가드 수정 src/components/common/HomePopup/HomePopupContainer.tsx	isLastGeneration 체크를 제거하여 HomePopup이 비-최신-생성 시나리오에서도 렌더링되도록 수정. period와 showOnlyToRecentGeneration 조건이 렌더링 판단의 주요 기준이 됨.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• feat: 홈 팝업 광고 리프레시 데이 이벤트 추가 #1901: HomePopupContainer.tsx의 HomePopup 표시 조건을 수정하는 동일한 파일 변경 (팝업 날짜 범위 조정)
• feat: 홈팝업 어드민 구좌 구현 #2016: HomePopupContainer.tsx의 렌더링 가드 로직을 변경하여 period와 showOnlyToRecentGeneration을 기반으로 렌더링 제어
• feat: 밸런스게임 모달 제거 #2152: HomePopupContainer.tsx의 HomePopup 렌더링 로직 수정으로 세대 및 날짜 확인 로직 변경

Suggested labels

size/M

Suggested reviewers

• imddoy

Poem

🐰 마지막 세대만의 굴레를 벗겨내고,
모든 세대가 팝업을 만날 수 있도록,
조건 맞춰 자유롭게 나타나는 HomePopup,
작은 수정이 큰 기쁨을 가져오네! ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/#2179

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

✨✨ 스토리북으로 확인하기 ✨✨

• Build
• Preview

[github-actions]

🚀 프리뷰 배포 확인하기 🚀

https://a7b4a9e6.sopt-internal-dev.pages.dev