Conversation
Summary of ChangesHello @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
🧠 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
Activity
Using Gemini Code AssistThe 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
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 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
|
| } | ||
|
|
||
| private func loadWebPage() { | ||
| guard let url = URL(string: "https://eatssu-coffee.figma.site/") else { return } |
EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
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 업데이트 |
EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift
Outdated
Show resolved
Hide resolved
EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift
Show resolved
Hide resolved
EATSSU/App/Sources/Presentation/Coffee/ViewController/CoffeeWebViewController.swift
Show resolved
Hide resolved
EATSSU/App/Sources/Presentation/Coffee/ViewController/CoffeeWebViewController.swift
Outdated
Show resolved
Hide resolved
EATSSU/App/Sources/Presentation/Coffee/ViewController/CoffeeWebViewController.swift
Show resolved
Hide resolved
- 커피 탭 setTab 호출 시 빈 화면 노출 방지 분기 처리 - 커피 웹뷰 중복 present 방지 가드 추가 - 다이얼로그 문자열 TextLiteral.Coffee로 이동 - 다이얼로그 레이아웃 SnapKit 제약조건으로 통일 - 커피 탭 불필요한 UINavigationController 래핑 제거
#️⃣ 관련 이슈
Resolved #392
💡작업 내용
💬리뷰 요구사항(선택)
3.2.1버전 병합 때문에 로그 이미지와 스플래시 뷰 관련 수정 사항이 있는데, 그건 무시하셔도 됩니다.