Skip to content

[Fix] 타임태그가 없을 때에도 상세페이지 클릭되도록 변경#59

Merged
oungsi2000 merged 1 commit intodevelopfrom
fix/58
Feb 24, 2026
Merged

[Fix] 타임태그가 없을 때에도 상세페이지 클릭되도록 변경#59
oungsi2000 merged 1 commit intodevelopfrom
fix/58

Conversation

@oungsi2000
Copy link
Contributor

@oungsi2000 oungsi2000 commented Feb 21, 2026

#️⃣ 이슈 번호

#58


🛠️ 작업 내용

  • 장소 미리보기 클릭(OnPlacePreviewClick) 이벤트 처리 시, 타임태그가 없는 축제일 경우에도 클릭이 되도록 수정했습니다

🙇🏻 중점 리뷰 요청

  • 특히 확인이 필요한 부분, 고민했던 부분 등을 적어주세요.

📸 이미지 첨부 (Optional)

Screen_Recording_20260221_210609_.Debug.Festabook.mp4

장소 미리보기 클릭(`OnPlacePreviewClick`) 이벤트 처리 시, 시간 태그(`selectedTimeTag`) 상태를 더 안전하게 확인하도록 로직을 수정했습니다.

- **`SelectEventHandler.kt` 수정:**
    - 기존의 `selectedTimeTag is LoadState.Success` 조건문 대신, `as?`를 사용한 안전한 타입 캐스팅으로 `timeTag` 값을 가져오도록 변경했습니다.
    - 이를 통해 `selectedTimeTag`가 `Success` 상태가 아닐 때도 이벤트 처리가 정상적으로 진행되고, 로깅 시 "undefined"로 기록되도록 개선했습니다.
@oungsi2000 oungsi2000 self-assigned this Feb 21, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 21, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/58

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

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

@oungsi2000 oungsi2000 merged commit d66cc68 into develop Feb 24, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants