[Fix] Firebase Remote Config 오류 수정#141
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces several updates to the authentication and networking layers, along with a new CI/CD configuration. Key changes include updating the Google Sign-In client ID, adding Crashlytics logging for login errors, and integrating a HostSelectionInterceptor into the network module. The SignUpResponse model was also updated to handle a list of identities. Feedback from the review suggests renaming the identity variable to its plural form for clarity, capturing dynamic error messages in Crashlytics instead of static strings, and refining the Bitrise workflow triggers to avoid redundant release builds on the development branch.
| @SerializedName("identity") | ||
| val identity: IdentityResponse | ||
| @SerializedName("identities") | ||
| val identity: List<IdentityResponse> |
There was a problem hiding this comment.
변수가 리스트 타입(List<IdentityResponse>)임에도 단수형인 identity로 명명되어 있습니다. JSON 키 값인 identities와 일치시키고 코드의 가독성을 높이기 위해 identities와 같이 복수형으로 변경하는 것을 권장합니다.
| val identity: List<IdentityResponse> | |
| val identities: List<IdentityResponse> |
There was a problem hiding this comment.
수정해주셔서 감사합니다. 해당 변경 사항 확인했습니다.
| ) | ||
| Firebase.crashlytics.setCustomKey("google_signin_phase", "get_credential") | ||
| Firebase.crashlytics.setCustomKey("google_signin_error_type", e.javaClass.simpleName) | ||
| Firebase.crashlytics.setCustomKey("google_signin_error_message", "A credential error occurred.") |
There was a problem hiding this comment.
Crashlytics에 기록되는 에러 메시지가 고정된 문자열로 설정되어 있습니다. 실제 발생한 예외 메시지(e.message)를 함께 기록하면 나중에 문제를 분석할 때 더 구체적인 정보를 얻을 수 있습니다.
| Firebase.crashlytics.setCustomKey("google_signin_error_message", "A credential error occurred.") | |
| Firebase.crashlytics.setCustomKey("google_signin_error_message", e.message ?: "A credential error occurred.") |
| triggers: | ||
| push: | ||
| - branch: "main" | ||
| - branch: "develop" |
There was a problem hiding this comment.
build-release 워크플로우가 develop 브랜치에서도 트리거되도록 설정되어 있습니다. 일반적으로 develop 브랜치는 build-qa 워크플로우에서 담당하므로, 의도된 것이 아니라면 develop 브랜치 트리거를 제거하여 빌드 자원 낭비를 방지하는 것이 좋습니다.
3 participants
#⃣ 연관된 이슈
close #140
📝 작업 내용
구글 콘솔과 파이어베이스를 함께 사용해서 발생한 문제 해결, 수정된 회원가입 api에 맞춰 body 값 보내도록 identity를 list 형태로 변경!