Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/#305 RuntimeError BugFix #306

Merged
merged 1 commit into from
May 14, 2024
Merged

Fix/#305 RuntimeError BugFix #306

merged 1 commit into from
May 14, 2024

Conversation

yuhaeun-la
Copy link
Contributor

작업내용

Appdelegate 코드 오류로 Firebase.configuration()이 두번 불리져서 런타임에러가 나는걸 확인했습니다.
코드 순서를 바꾸고 확실하게 release일때만 configureFirebase()함수가 불릴 수 있도록 수정했습니다

리뷰요청

관련 이슈

close #305

@yuhaeun-la yuhaeun-la added the 🐞버그픽스 버그 수정 label May 13, 2024
@yuhaeun-la yuhaeun-la added this to the v1.3.0 milestone May 13, 2024
@yuhaeun-la yuhaeun-la requested a review from MUKER-WON May 13, 2024 18:56
@yuhaeun-la yuhaeun-la self-assigned this May 13, 2024
@MUKER-WON
Copy link
Contributor

debug일 때 configureFirebase 가 실행안되게 되는데 해당 부분은 상관없을까요..?
현재 저 filePath가 노티 받는 firebase라 debug일 때 없어도 될까요

@MUKER-WON
Copy link
Contributor

아 노티 받는데 이상있지는 않네요
GoogleService-debugging-Info를 가지고 있는 파베 콘솔은 하은님이 따로 만드신 거죠?

@yuhaeun-la
Copy link
Contributor Author

아 노티 받는데 이상있지는 않네요 GoogleService-debugging-Info를 가지고 있는 파베 콘솔은 하은님이 따로 만드신 거죠?

넵 그렇습니다

@yuhaeun-la
Copy link
Contributor Author

yuhaeun-la commented May 14, 2024

아 노티 받는데 이상있지는 않네요 GoogleService-debugging-Info를 가지고 있는 파베 콘솔은 하은님이 따로 만드신 거죠?

참고로 어제 데이터쪽에 파베 애널리틱스 가져가게 하려고 별 방법을 다 써봤는데 제가 구조 이해가 부족한건지 코드 상으로 뭐가 부족했던건지 잘 안돼서 ㅠㅠ 일단 적용 부분은 같이 얘기해봐야할듯해요

@MUKER-WON
Copy link
Contributor

고생하셨습니다!!

@MUKER-WON MUKER-WON merged commit 5bbb894 into dev May 14, 2024
1 check passed
@MUKER-WON MUKER-WON deleted the fix/#305 branch May 14, 2024 08:02
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
Labels
🐞버그픽스 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 함수가 두번 불려 런타임 에러가 나는 버그
2 participants