Feat/#302 FirebaseAnalytics 작업 #307
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 tasks
gnksbm
reviewed
May 17, 2024
Comment on lines
51
to
56
| func configureDebuggingFB(application: UIApplication) { | ||
| guard let filePath = Bundle.main.path( | ||
| forResource: "GoogleService-debugging-Info", | ||
| forResource: "GoogleService-Info-debugging", | ||
| ofType: "plist" | ||
| ), | ||
| let options = FirebaseOptions(contentsOfFile: filePath) |
There was a problem hiding this comment.
Suggested change
| func configureDebuggingFB(application: UIApplication) { | |
| guard let filePath = Bundle.main.path( | |
| forResource: "GoogleService-debugging-Info", | |
| forResource: "GoogleService-Info-debugging", | |
| ofType: "plist" | |
| ), | |
| let options = FirebaseOptions(contentsOfFile: filePath) | |
| func configureFirebase(application: UIApplication) { | |
| var googleInfoName: String | |
| #if DEBUG | |
| googleInfoName = "GoogleService-Info" | |
| #else | |
| googleInfoName = "GoogleService-Info-debugging" | |
| #endif | |
| guard let filePath = Bundle.main.path( | |
| forResource: googleInfoName, | |
| ofType: "plist" | |
| ), | |
| let options = FirebaseOptions(contentsOfFile: filePath) | |
| else { return } | |
| FirebaseApp.configure(options: options) | |
| application.registerForRemoteNotifications() | |
| } |
기존의 함수에서 전처리문으로 리소스 경로만 수정하는게 어떨까요?
There was a problem hiding this comment.
이 버전 코드가 앱 델리게이트쪽 코드를 더 깔끔하게 해줄 것 같네요 수정하도록 하겠습니다 ~
Comment on lines
26
to
28
| // var newArguments = ProcessInfo.processInfo.arguments | ||
| // newArguments.append("-FIRDebugEnabled") | ||
| // ProcessInfo.processInfo.setValue(newArguments, forKey: "arguments") |
There was a problem hiding this comment.
이부분도 가독성을 위해 삭제하도록 하겠습니다 ~
| @@ -9,6 +9,7 @@ | |||
| import Foundation | |||
|
|
|||
| import RxSwift | |||
| import FirebaseAnalytics | |||
yuhaeun-la
added a commit
that referenced
this pull request
May 17, 2024
* [Feat] 2x2 4x2 Widget Base작업 (#244) * [add] 2x2 ArrivalInfoView * [Chore] #238 Extension + NanumFont for Convenience * [Feat] ArrivalInfoSmallView UI작업 * [Add] BusStopOptionProvider 주석 제거 * [UI] Widget Background ColorChange * [Add] AppIntent 수정 * [add] ArrivalInfoIntent 코드 수정 * [Feat] ArrivalInfoMediumView * [Feat] ArrivalRemaining 값 추가, ArrivalState에 따른 Color 변경 추가 * [Chore] Code Convention * [UI] Font Size 수정 * Update ArrivalInfoIntent.swift * [Setting] Tuist Widget 타겟 추가 * [chore] Delete @parameter * [Delete] file - --------- Co-authored-by: Geonseob Kim <[email protected]> * [Fix] SearchView 주변정류장 업데이트 시점 변경 * [Docs] 사진 업데이트, 문구 살짝 수정 * Feat/#263 UI변경작업 (#269) * [add] pictures + * [UI] SplashView 변경 * [UI] Image Upload + Change Icon Size * [Setting] Lottie * [Setting] Lottie Project file + Lottie extension code * [Setting] Change Setting + Update Lottie * [UI] EmptyFavoriteView 변경 작업 * [UI] Change BusStopInformationView + SearchView * [UI] BusStopInformationView Update * [Chore] setting * [UI] 정규 알람뷰 수정 * [UI] 색감 통일 * [UI] SearchBar 색 변경 * [Add] Lottie 이미지 renderingEngine을 mainThread로 지정 * [chore] 안쓰는 파일 delete * [Perf] 함수 수정으로 코드 반복 줄임 * [UI] searchView 글자크기수정 * [Chore] BusStopView의 refreshBtn Color 수정 및 textField 제약조건 미세 수정 * [Design] Search된 TableView Background Color 수정 --------- Co-authored-by: MUKER-WON <[email protected]> * [UI] BusStopView및 nearMapView수정 * [UI] Color change * [UI] UI 변경 * [UI] progressColor, refreshColor Change * [UI] Icon 변경 * [add] Update LeafMarker * [UI] 배경 변경 전 최종 * [UI] 정말 많은 것을 변경했습니다 - 스플래쉬 뷰, 아이콘 변경 * [Feat] 다크모드 추가 * [UI] 정기 알람뷰 * [Feat] 다크모드 보완 * [UI] RegularAlarmView weekdaButton 색 수정 * [UI] 소소한 UI 수정 * Revert "UI수정 UI수정작업 " (#274) * Revert "Revert "UI수정 UI수정작업 " (#274)" (#278) This reverts commit e57079e. * UI/#279 네비게이션 위치 통일 * [Bug] 잔상 버그 수정 (#282) * Bug/#283 NearMapView 위치정보권한 없을 때 나오는 infomationView 수정 (#284) * [Chore] 레거시 코드 삭제 * [Fix] 에러처리 수정 * [Fix] error 분기 처리 * [Fix] 레거시 코드 삭제 * [Fix] 레거시 코드 삭제 * [Fix] 레거시 코드 삭제 * Fix/#287 NearMapView에 InformainView 세브론UI 설정 (#288) * [Design] NearMapView InformationView 세브론UI 추가 * [Chore] 의미없는 코드 * Fix/#285 설정 Icon 떴다 안떴다 하는 버그 및 UI수정 (#289) * [Fix] SettingView 아이콘 가끔 떴다 안떴다 하는 문제 해결 * [chore] 다크모드 UI 수정 및 legacy코드 제거 * [chore] legacy code 삭제 * [UI] Color add * [UI] Tabbar Color Change * [Fix] distance에 '알수없음'이 들어오는 로직 수정 (#293) * UI/#291 Cell에 text 흘러넘치는 UI오류 (#295) * [UI] #291 긴 text가 label 범위를 벗어나는 UI 수정 * [Fix] #291 NearMapInformationView text scale factor 활성화 코드 수정 * UI/#294 가독성을 위해 컬러 변경 (#296) * [UI] 색상변경 * [fix] 위치허용 '알수없음' 뜨는 버그 해결 * [chore] code - * [Fix] EmptyFavoriteView 삭제된 글귀 업데이트 * [UI] 글자 가독성 추가 * [chore] for convention * [Fix] #297 즐겨찾기 위치 수정 (#298) * ReleaseFix/#300 v1.2.2 배포를 위한 Setting 수정 (#301) * [Fix] 1.2.2 버전 오류 수정 * [Setting] testflight 수정 * [Fix] Tuist 수정 * [Fix] Fastlane 파일 수정 * [Docs] README 업데이트 * [Docs] 버전 및 다크모드 업데이트 * Setting/#303 구글 애널리틱스 환경 세팅 (#304) * [Setting] #303 파이어베이스 환경 세팅 * [Setting] project.stencil 파일 변경 /템플릿 변경 및 DependcyPlugin추가) * [setting] #303 project.swift파일에 scheme 추가 * [Setting] stencil 제거 및 Scheme파일 수정 * [Setting]#303 잘못삭제한 코드 제거 및 스킴추가 * [Setting] Debug모드 활성화를 위한 코드 추가 * [chore] #303 불필요한 코드 제거 * [Setting] #303 FirebaseAnalytics package추가 후에 변경 필요할지도 * [chore] #303 legacy 코드 제거 * [fix] #305 Appdelegate 수정 (#306) * Feat/#302 FirebaseAnalytics 작업 (#307) * [Setting] .networkService 추가 * [Setting] .networkService 제거 및 Data/Project.swift에 .package추가 * [chore] +package 재추가 * [chore] 내 너무 힘들다 레전드로 마음이 힘듦 * [Setting] 릴리즈 모드 디버그 모드 구분 * [Fix] 애널릭티스 의존 변경 * [setting] #302 Makefile 및 Appdelegate+Firebase.swift수정 * [Add] #302 app모듈 Scheme 추가 * [chore] Data.swift package삭제 * [UI] 색 통일성 * [UI] 소소한 UI 수정 * [perf] #302 make파일 피드백 반영 및 download 리스트에서 googleinfo2 제거 * [Perf] #302 피드백 반영 * [chore] 코드 컨벤션 및 레거시 코드 삭제 * [Fix] #302 파일 네임 수정 * [setting] 버전 변경 1.2.2 -> 1.2.3 --------- Co-authored-by: MUKER-WON <[email protected]> --------- Co-authored-by: Geonseob Kim <[email protected]> Co-authored-by: Geonseob Kim <[email protected]> Co-authored-by: Kang Muk <[email protected]> Co-authored-by: Jisoo Ham <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업내용
리뷰요청
관련 이슈
close #302