Skip to content

fix: MainViewController toolTipView 문구 수정#794

Merged
Do-hyun-Kim merged 1 commit into
developfrom
feat/#793-main-tooltip-remove
Feb 19, 2026
Merged

fix: MainViewController toolTipView 문구 수정#794
Do-hyun-Kim merged 1 commit into
developfrom
feat/#793-main-tooltip-remove

Conversation

@Do-hyun-Kim

Copy link
Copy Markdown
Collaborator

🔵PR을 올리기 전 아래 사항을 확인해주세요.

  • 구현한 로직과 기능이 올바르게 작동되는지 충분히 테스트해주세요.
  • 코드의 성능이나 메모리 효율성이 적절하게 고려되었는지, 불필요한 코드가 없는지 검토해주세요.
  • 이번 PR에서 구현된 주요 기능이나 해결된 문제에 대해 자세히 서술해주세요.
    (위 내용은 지워주세요)

😽개요

  • 생존신고 활성화 시 balloonTextType을 survivalStandard로 수정해요

🛠️작업 내용

  • 생존신고 활성화 될 경우 MainViewReactor내부 함수 중 setBalloonText(_:)survivalStandard Type으로 변경하여 문구를 변경했습니다.

🟡차후 계획 (Optional)

  • MainViewReactor이전에 수정하면서 느낀 거긴 한데 너무 로직이 하나의 Reactor에 밀집되어 있다 보니 복잡도가 올라가 있는 것 같아서 조금 Reactor를 분리하거나, Interface 단위로 분리하는 방안으로 쪼개면 좋을 것 같음 (개인적인 생각)

✅테스트 케이스

  • 생존 신고 가능 시간에 Tooltip 문구가 기획 의도대로 뜨는지 확인해요.

🙏🏻아래와 같이 PR을 리뷰해주세요.

  • PR 내용이 부족하다면 보충 요청해주세요.
  • 코드 스타일이 팀의 규칙에 맞게 작성되었는지, 일관성을 유지하고 있는지 확인해주세요.
  • 코드에 대한 문서화나 주석이 필요한 부분에 적절하게 작성되어 있는지 확인해주세요.
  • 구현된 로직이 효율적이고 올바르게 작성되었는지, 아키텍처를 잘 준수하고 있는지 검토해주세요.
  • 네이밍, 포매팅, 주석 등 코드의 일관성이 유지되고 있는지 확인해주세요.

- Project+Templates 1.4.4 버전으로 수정

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the tooltip text shown on the main view when the survival report feature is active. The issue was that the balloon was incorrectly showing midnight-specific text ("00:00:00 후 잠금 해제") during normal hours. The fix changes the balloon text type from midNightStandard to survivalStandard, which displays the more appropriate message "하루에 한 번 사진을 올릴 수 있어요" (You can upload a photo once a day). The version is also bumped from 1.4.3 to 1.4.4 to reflect this bug fix.

Changes:

  • Changed balloon text type from midNightStandard to survivalStandard in MainViewReactor's setBalloonText function
  • Bumped app version from 1.4.3 to 1.4.4 in project configuration files

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
14th-team5-iOS/Bibbi/Sources/Presentation/Home/Reactor/MainViewReactor.swift Fixed balloon text to use survivalStandard instead of midNightStandard for the default survival page state
Tuist/ProjectDescriptionHelpers/Project+Templates.swift Updated MARKETING_VERSION from 1.4.3 to 1.4.4
14th-team5-iOS/Bibbi/Project.swift Updated CFBundleShortVersionString from 1.4.3 to 1.4.4

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Do-hyun-Kim Do-hyun-Kim merged commit aa97199 into develop Feb 19, 2026
6 of 7 checks passed
@Do-hyun-Kim Do-hyun-Kim deleted the feat/#793-main-tooltip-remove branch February 19, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants