Skip to content

Conversation

@ikseong00
Copy link
Collaborator

@ikseong00 ikseong00 commented Nov 22, 2025

Related issue 🛠

Work Description 📝

  • 화면 표시 상단바, 하단바 가려지는 현상 수정
  • 네이버 지도 키 입력 관련 코드 수정

Summary by CodeRabbit

릴리스 노트

  • 개선사항
    • 지도 서비스 초기화가 통합되어 지도의 안정성과 로딩 신뢰도가 향상되었습니다.
    • 상태/내비게이션 바와의 레이아웃 호환성이 개선되어 다양한 기기에서 화면 표시가 더 나아졌습니다.
    • 네이티브 라이브러리 처리 방식이 변경되어 런타임 안정성이 향상되었습니다.
  • 업데이트
    • 앱 버전 업데이트: 1.0.4 (버전 코드 5)

✏️ Tip: You can customize this high-level summary in your review settings.

Naver Map SDK 사용을 위한 초기화 코드를 Application 클래스에 추가합니다.
@coderabbitai
Copy link

coderabbitai bot commented Nov 22, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Kakao SDK 초기화를 BuildConfig 키로 변경하고 setKakao() 래퍼를 제거해 onCreate에서 직접 초기화하며, Naver Map SDK NcpKeyClient 초기화를 추가했다. 테마에 android:fitsSystemWindows="true"를 추가하고 AndroidManifest에 android:extractNativeLibs="true"를 설정했으며 앱 버전과 카메라 라이브러리 버전이 상향되었습니다.

Changes

Cohort / File(s) Summary
앱 초기화 코드
app/src/main/java/com/kuit/findu/FindUApp.kt
KakaoSdk.init 호출이 KAKAO_NATIVE_APP_KEY 대신 BuildConfig.KAKAO_NATIVE_APP_KEY 사용으로 변경. setKakao() 래퍼 제거 후 onCreate에서 직접 초기화. Naver Map SDK 초기화 추가: NaverMapSdk.getInstance(this).client = NaverMapSdk.NcpKeyClient(BuildConfig.NAVER_CLIENT_ID). 관련 import 조정.
테마 조정
app/src/main/res/values/themes.xml
Base.Theme.FindU 스타일에 android:fitsSystemWindows="true" 속성 추가.
매니페스트 변경
app/src/main/AndroidManifest.xml
<application> 요소에 android:extractNativeLibs="true" 속성 추가.
버전 업데이트
app/build.gradle.kts
versionCode25 로, versionName"1.0.1""1.0.4" 로 변경.
라이브러리 버전 조정
gradle/libs.versions.toml
[versions] 항목의 cameraCore 버전을 "1.1.0-beta01""1.4.1" 로 상향.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant App as FindUApp.onCreate
  participant Kakao as KakaoSdk
  participant Naver as NaverMapSdk
  Note over App: 앱 시작 — SDK 초기화 순서
  App->>Kakao: KakaoSdk.init(BuildConfig.KAKAO_NATIVE_APP_KEY)
  activate Kakao
  Kakao-->>App: 초기화 완료
  deactivate Kakao
  App->>Naver: NaverMapSdk.getInstance(this).client = NcpKeyClient(BuildConfig.NAVER_CLIENT_ID)
  activate Naver
  Naver-->>App: 클라이언트 설정 완료
  deactivate Naver
  Note right of App: 완료 — SDKs 초기화 및 앱 실행 준비
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • 파일 수는 적으나 초기화 순서·키 소스(BuildConfig)·매니페스트 패키징 설정과 라이브러리 버전 상향이 섞여 있어 각 항목 별 확인 필요.
  • 리뷰 시 중점 확인 항목:
    • BuildConfig.KAKAO_NATIVE_APP_KEYBuildConfig.NAVER_CLIENT_ID가 빌드 설정에 정확히 존재하는지.
    • NaverMapSdk NcpKeyClient 사용법과 import가 맞는지(권한/401 원인 해결 여부).
    • android:fitsSystemWindows="true" 적용으로 인한 레이아웃 영향(상단/하단 여백).
    • android:extractNativeLibs="true"가 패키징·릴리스 정책에 미치는 영향.
    • cameraCore 버전 상향으로 인한 호환성/추가 API 변경 여부.

Possibly related PRs

Poem

작은 키로 문을 열고 SDK들이 모였네 🌱
지도 하나, 카카오 하나, 시작의 손길로
상단과 하단의 빈틈 채워 넣고
버전은 살짝 올라가며 웃음 짓네 🚀
빌드가 끝나면 새로운 길이 보이리 ✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경사항인 화면 표시 범위 수정과 네이버맵 의존성 수정을 명확하게 요약하고 있습니다.
Description check ✅ Passed PR 설명이 관련 이슈, 작업 내용을 포함하고 있지만 선택적 섹션(스크린샷, 완료되지 않은 작업, 리뷰어 의견)이 누락되어 있습니다.
Linked Issues check ✅ Passed 코드 변경이 연결된 이슈 #105의 목표를 충족합니다: fitsSystemWindows 추가로 화면 표시 범위 수정, KakaoSdk와 NaverMapSdk 초기화로 401 에러 해결.
Out of Scope Changes check ✅ Passed 모든 변경사항이 이슈 #105의 범위 내에 있습니다. 버전 업데이트와 라이브러리 추출 설정은 배포 관련 필수 변경사항입니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between adcb614 and 5dc41ad.

📒 Files selected for processing (2)
  • app/build.gradle.kts (1 hunks)
  • gradle/libs.versions.toml (1 hunks)

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 merged commit 225e650 into develop Nov 26, 2025
1 check passed
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.

[fix] 화면 표시 범위, 네이버맵 의존성 수정

2 participants