Skip to content

[#522] 로그아웃 시 미등록 FCM 토큰 삭제 실패로 로그아웃이 중단되는 문제를 해결한다#523

Merged
opficdev merged 4 commits into
developfrom
fix/#522-fcm-logout
Jun 2, 2026
Merged

[#522] 로그아웃 시 미등록 FCM 토큰 삭제 실패로 로그아웃이 중단되는 문제를 해결한다#523
opficdev merged 4 commits into
developfrom
fix/#522-fcm-logout

Conversation

@opficdev
Copy link
Copy Markdown
Owner

@opficdev opficdev commented Jun 2, 2026

🔗 연관된 이슈

🎯 의도

  • GitHub 로그인/로그아웃 과정에서 FCM 토큰 등록과 OAuth redirect 설정이 정상 반영되지 않던 문제 해결

📝 작업 내용

📌 요약

  • 앱 타깃에 APNs 환경 설정 추가
  • 앱 전용 App.xcconfig를 추가해 로컬 Config.xcconfig 값을 Info.plist 치환에 사용
  • 로그인 버튼의 탭 가능 영역이 텍스트에만 잡히던 문제 수정

🔍 상세

  • APS_ENVIRONMENT를 Debug는 development, Release는 production으로 설정
  • Application/DevLogApp/Sources/App.xcconfig에서 공통 version 설정과 로컬 config 설정을 함께 include
  • .gitignore 대상인 Config.xcconfig는 optional include로 처리해 CI 환경에서 파일 누락으로 인한 설정 로드 실패 방지
  • 빌드 산출물 기준으로 APP_REDIRECT_URL, GITHUB_CLIENT_ID, GIDClientID, URL scheme 치환 확인
  • 빌드 산출물 기준으로 aps-environment = development 반영 확인
  • LoginButton의 frame/overlay를 label 내부로 이동해 버튼 전체 영역이 탭되도록 수정

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this Jun 2, 2026
Copy link
Copy Markdown
Contributor

@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

이번 풀 리퀘스트에서는 새로운 App.xcconfig 설정 파일을 추가하고, 프로젝트 설정(Project.swift 및 Xcode 프로젝트 파일)을 업데이트하여 APS_ENVIRONMENT 빌드 설정을 개발(development) 및 운영(production) 환경에 맞게 구성했습니다. 또한, LoginButton 뷰의 레이아웃 구조를 개선하였습니다. 리뷰 의견으로는 LoginButton에서 .contentShape(.capsule)을 버튼 외부에 적용할 경우 투명한 영역에서 터치 제스처가 정상적으로 감지되지 않을 수 있으므로, 이를 버튼 label 내부의 Text 뷰 뒤로 이동할 것을 제안합니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread Application/DevLogPresentation/Sources/Common/Component/LoginButton.swift Outdated
@opficdev opficdev merged commit beadf3e into develop Jun 2, 2026
1 check passed
@opficdev opficdev deleted the fix/#522-fcm-logout branch June 2, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

로그아웃 시 미등록 FCM 토큰 삭제 실패로 로그아웃이 중단되는 문제를 해결한다

1 participant