Skip to content

[#392] 커피 이벤트 추가 #393

Open
Hrepay wants to merge 9 commits intodevelopfrom
feat/#392
Open

[#392] 커피 이벤트 추가 #393
Hrepay wants to merge 9 commits intodevelopfrom
feat/#392

Conversation

@Hrepay
Copy link
Member

@Hrepay Hrepay commented Feb 26, 2026

#️⃣ 관련 이슈

Resolved #392

💡작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 바텀 네비게이션에 커피 이벤트(나만아니면돼~) 탭 추가 및 웹뷰 연동
  • 커피 웹뷰를 전체화면 모달 방식으로 변경
  • 웹뷰 닫기 시 이탈 확인 다이얼로그 추가
  • 커피 탭 아이콘 이미지 교체
  • 커피 이벤트 화면 ID(ABM_001000) 추가

Simulator Screen Recording - iPhone 13 mini - 2026-02-26 at 21 43 58

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • 3.2.1버전 병합 때문에 로그 이미지와 스플래시 뷰 관련 수정 사항이 있는데, 그건 무시하셔도 됩니다.

@Hrepay Hrepay requested a review from wjdalswl February 26, 2026 12:49
@Hrepay Hrepay self-assigned this Feb 26, 2026
Copilot AI review requested due to automatic review settings February 26, 2026 12:49
@gemini-code-assist
Copy link

Summary of Changes

Hello @Hrepay, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 사용자에게 새로운 '커피 이벤트' 기능을 제공하기 위해 바텀 네비게이션 바에 전용 탭을 추가합니다. 이 탭을 통해 접근하는 커피 이벤트는 전체 화면 모달 웹뷰로 구현되었으며, 사용자가 웹뷰를 닫으려 할 때 진행 상황 저장 여부를 묻는 확인 다이얼로그를 표시하여 의도치 않은 이탈을 방지합니다. 또한, 새로운 탭에 맞는 아이콘과 Firebase 화면 ID가 추가되었으며, 기존 스플래시 화면의 크리스마스 테마 이미지가 일반 로고로 복원되었습니다.

Highlights

  • 새로운 '커피 이벤트' 탭 추가: 바텀 네비게이션에 새로운 '커피 이벤트' 탭이 추가되었습니다.
  • 전체 화면 모달 웹뷰 구현: 커피 이벤트는 전체 화면 모달 웹뷰로 표시되며, 웹뷰 종료 시 이탈 확인 다이얼로그가 추가되었습니다.
  • 탭 아이콘 및 Firebase ID 업데이트: 새로운 커피 탭 아이콘 이미지가 적용되었고, Firebase에 커피 이벤트 화면 ID(ABM_001000)가 추가되었습니다.
  • 스플래시 화면 이미지 복원: 스플래시 화면의 크리스마스 테마 이미지가 기본 로고로 변경되었습니다. (리뷰 요청에 따라 이 부분은 참고용입니다.)

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • EATSSU/App/Resources/Assets/Images.xcassets/AppIcon_christmas.appiconset/Contents.json
    • 크리스마스 테마 앱 아이콘 에셋 구성이 추가되었습니다.
  • EATSSU/App/Sources/Data/Firebase/FirebaseScreenID.swift
    • 커피 이벤트(AnyoneButMeActivity)를 위한 ABM_001000 Firebase 화면 ID가 추가되었습니다.
  • EATSSU/App/Sources/Presentation/Coffee/ViewController/CoffeeWebViewController.swift
    • 커피 이벤트를 표시하는 CoffeeWebViewController가 새로 추가되었으며, 웹뷰, 닫기 버튼, 이탈 확인 다이얼로그 로직을 포함합니다.
  • EATSSU/App/Sources/Presentation/Splash/SplashViewController.swift
    • 스플래시 화면에서 크리스마스 테마 배경 이미지를 비활성화하고 로고 이미지를 기본 로고로 변경했습니다.
  • EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift
    • Tab 열거형에 coffee 탭이 추가되고 기존 탭들의 인덱스가 조정되었습니다.
    • tabViewControllers 배열에 커피 탭을 위한 UIViewController 플레이스홀더가 추가되었습니다.
    • 새로운 커피 탭의 아이콘(coffee, coffeeSelected) 구성이 추가되었습니다.
    • 커피 웹뷰를 전체 화면 모달로 표시하는 presentCoffeeWebView 메서드가 추가되었습니다.
    • tabBarController(_:shouldSelect:) 델리게이트 메서드에서 커피 탭 선택 시 presentCoffeeWebView를 호출하도록 로직이 추가되었습니다.
    • tabBarController(_:didSelect:) 델리게이트 메서드에 커피 탭 선택 시 처리가 추가되었습니다.
  • EATSSU/App/Sources/Utility/Literal/TextLiteral.swift
    • 커피 탭의 텍스트 리터럴 '나만아니면돼~'가 추가되었습니다.
  • EATSSUDesign/EATSSUDesign/Resources/Images.xcassets/coffee.imageset/Contents.json
    • 커피 탭의 기본 아이콘 에셋 구성이 추가되었습니다.
  • EATSSUDesign/EATSSUDesign/Resources/Images.xcassets/coffee_selected.imageset/Contents.json
    • 커피 탭의 선택된 상태 아이콘 에셋 구성이 추가되었습니다.
Activity
  • PR 작성자가 버전 3.2.1 병합으로 인한 로그 이미지 및 스플래시 뷰 관련 수정 사항은 리뷰 시 무시해도 된다고 언급했습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.


새로운 탭 열려, 커피 향기 가득해라, 모달 창에 띄워.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

이번 PR은 커피 이벤트 탭 추가, 웹뷰 연동, 모달 방식 변경, 종료 다이얼로그 추가, 아이콘 교체, Firebase Screen ID 추가 등 여러 기능을 성공적으로 구현했습니다. 전반적으로 잘 구현되었지만, 몇 가지 개선 사항이 있습니다.

}

private func loadWebPage() {
guard let url = URL(string: "https://eatssu-coffee.figma.site/") else { return }

Choose a reason for hiding this comment

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

medium

웹뷰 URL을 Constants 파일이나 Info.plist를 통해 관리하여 유연성을 높이는 것을 권장합니다.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

이 PR은 이슈 #392에 맞춰 바텀 탭바에 “나만아니면돼~(커피 이벤트)” 탭을 추가하고, 해당 탭에서 커피 이벤트 웹뷰를 전체화면 모달로 노출하며 이탈 확인 다이얼로그/분석 ScreenID/아이콘 리소스를 함께 반영합니다.

Changes:

  • 탭바에 커피 이벤트 탭 추가 및 탭 탭 시 전체화면 웹뷰 모달 표시
  • 웹뷰 닫기 시 이탈 확인 다이얼로그 추가, Firebase ScreenID(ABM_001000) 추가
  • 커피 탭 아이콘/일부 앱 아이콘(크리스마스 포함) 리소스 업데이트

Reviewed changes

Copilot reviewed 8 out of 48 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift 커피 탭 추가 및 탭 선택 시 웹뷰 모달 present 로직 반영
EATSSU/App/Sources/Presentation/Coffee/ViewController/CoffeeWebViewController.swift 커피 이벤트 웹뷰 + 닫기 버튼 + 이탈 확인 다이얼로그 구현
EATSSU/App/Sources/Utility/Literal/TextLiteral.swift 탭바 타이틀(커피) 문자열 추가
EATSSU/App/Sources/Data/Firebase/FirebaseScreenID.swift 커피 이벤트 ScreenID(ABM_001000) 추가
EATSSU/App/Sources/Presentation/Splash/SplashViewController.swift 스플래시 로고 리소스 변경(주석 처리 코드 포함)
EATSSUDesign/.../coffee*.imageset/* 커피 탭 아이콘(선택/미선택) 리소스 추가
EATSSU/App/Resources/Assets/Images.xcassets/AppIcon*.appiconset/* 앱 아이콘 리소스/Contents.json 업데이트

- 커피 탭 setTab 호출 시 빈 화면 노출 방지 분기 처리
- 커피 웹뷰 중복 present 방지 가드 추가
- 다이얼로그 문자열 TextLiteral.Coffee로 이동
- 다이얼로그 레이아웃 SnapKit 제약조건으로 통일
- 커피 탭 불필요한 UINavigationController 래핑 제거
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.

[Feat] 이벤트 탭 추가

2 participants