Skip to content

[fix] #99 카카오톡 미설치 기기에서 카카오 로그인 진행되지 않는 현상 수정#100

Merged
Ojongseok merged 1 commit into
developfrom
fix/#99-kakao-login
Feb 10, 2026
Merged

[fix] #99 카카오톡 미설치 기기에서 카카오 로그인 진행되지 않는 현상 수정#100
Ojongseok merged 1 commit into
developfrom
fix/#99-kakao-login

Conversation

@Ojongseok
Copy link
Copy Markdown
Member

@Ojongseok Ojongseok commented Feb 10, 2026 •
edited by coderabbitai Bot
Loading

🔗 관련 이슈

📙 작업 설명

  • local.properties에서 조회한 키를 manifestPlaceholders에 저장하는 로직 수정

🧪 테스트 내역 (선택)

  • 카카오톡 미설치된 에뮬레이터 작동 확인
  • 모바일 작동확인

📸 스크린샷 또는 시연 영상 (선택)

기능 미리보기 기능 미리보기
기능 설명

💬 추가 설명 or 리뷰 포인트 (선택)

  • local.properties에 따옴표 ""를 제거하면 trim('"')을 제거할 수 있으나 다른 local.properties 내 키들과 일관된 형태로 작성하기 위해 ""를 제거하지 않았습니다.

Summary by CodeRabbit

  • Chores
    • 빌드 구성 설정을 최적화했습니다.

@Ojongseok Ojongseok requested a review from ikseong00 February 10, 2026 12:59
@Ojongseok Ojongseok self-assigned this Feb 10, 2026
@Ojongseok Ojongseok changed the title [fix] #99 카카오 로그인 진행되지 않는 현상 수정 [fix] #99 카카오톡 미설치 기기에서 카카오 로그인 진행되지 않는 현상 수정 Feb 10, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 10, 2026 •
edited
Loading

Walkthrough

app/build.gradle.kts에서 KAKAO_NATIVE_APP_KEY와 NAVER_MAP_CLIENT_ID 설정이 로컬 변수로 리팩토링되었습니다. KAKAO_NATIVE_APP_KEY는 manifestPlaceholders에 전달되기 전에 따옴표를 제거하도록 처리되었으며, buildConfigField 설정은 그대로 유지됩니다.

Changes

Cohort / File(s) Summary
빌드 설정 리팩토링
app/build.gradle.kts		 KAKAO_NATIVE_APP_KEY와 NAVER_MAP_CLIENT_ID를 로컬 변수로 추출하고, KAKAO_NATIVE_APP_KEY의 따옴표를 제거한 값을 manifestPlaceholders에 전달하도록 수정

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 카카오 로그인의 길을 막던 따옴표,
토끼가 슥삭 깎아내니 앱으로 돌아오네!
빌드 설정 정리하면, 사용자 행복도 함께 상승 📱✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 카카오 로그인 관련 버그 수정을 명확히 설명하고 있으며, 실제 변경 내용(카카오 로그인 미설치 기기에서의 현상 수정)과 일치합니다.
Linked Issues check ✅ Passed 이슈 #99는 카카오 로그인 후 앱으로 복귀되지 않는 버그 수정을 요구하며, PR은 manifestPlaceholders의 키 저장 로직 수정으로 이를 해결하고자 합니다.
Out of Scope Changes check ✅ Passed PR의 모든 변경 사항(app/build.gradle.kts의 KAKAO_NATIVE_APP_KEY 처리 로직)이 이슈 #99의 카카오 로그인 버그 수정과 직접 관련되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#99-kakao-login

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
app/build.gradle.kts (1)

25-26: properties에 키가 없을 경우 "null" 문자열이 할당됩니다.

properties["KEY"]가 null을 반환할 때 .toString()은 문자열 "null"이 됩니다. local.properties에 키가 누락된 경우 빌드 시 명확한 에러 없이 잘못된 값이 사용될 수 있습니다. 기존 코드에서도 동일한 문제가 있지만, 로컬 변수로 리팩토링한 김에 방어 처리를 추가하면 좋습니다.

🛡️ null 방어 처리 제안 
-        val naverMapClientId = properties["NAVER_MAP_CLIENT_ID"].toString()
-        val kakaoKey = properties["KAKAO_NATIVE_APP_KEY"].toString()
+        val naverMapClientId = requireNotNull(properties["NAVER_MAP_CLIENT_ID"]) {
+            "NAVER_MAP_CLIENT_ID is missing in local.properties"
+        }.toString()
+        val kakaoKey = requireNotNull(properties["KAKAO_NATIVE_APP_KEY"]) {
+            "KAKAO_NATIVE_APP_KEY is missing in local.properties"
+        }.toString()

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

ikseong00
ikseong00 approved these changes Feb 10, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@ikseong00 ikseong00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

" 😢

@Ojongseok Ojongseok merged commit 4018578 into develop Feb 10, 2026
4 checks passed
@Ojongseok Ojongseok deleted the fix/#99-kakao-login branch February 10, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ikseong00 ikseong00 ikseong00 approved these changes

Assignees

@Ojongseok Ojongseok

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[fix] 카카오톡 미설치 기기에서 로그인 진행되지 않는 현상 수정

2 participants

@Ojongseok @ikseong00