Skip to content

AndroidManifest.xml からserviceを削除#5151

Merged
TinyKitten merged 1 commit intodevfrom
TinyKitten-patch-1
Jan 25, 2026
Merged

AndroidManifest.xml からserviceを削除#5151
TinyKitten merged 1 commit intodevfrom
TinyKitten-patch-1

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Jan 25, 2026

#5133

Summary by CodeRabbit

リリースノート

  • バグ修正
    • バックグラウンド位置情報サービスの登録を削除しました。これにより、アプリケーションの安定性が向上します。

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 25, 2026

📝 Walkthrough

ウォークスルー

AndroidManifest.xmlから、expo.modules.location.servicesの場所サービスタスク宣言を削除しました。この変更により、フォアグラウンド位置情報サービスの明示的なサービス登録が削除されます。

変更内容

コホート / ファイル 変更概要
Androidマニフェスト設定
android/app/src/main/AndroidManifest.xml
expo.modules.location.services.LocationTaskServiceのサービス宣言を削除(android:foregroundServiceType="location|mediaPlayback"およびtools:node="merge"属性を含む)

推奨される関連PR

レビュー工数の見積もり

🎯 1 (Trivial) | ⏱️ ~2分

推奨ラベル

android

ポエム

🐰 うさぎが走る、位置情報なき道
マニフェストから消えた宣言
フォアグラウンド、さようなら!
シンプルなコードは美しい 🌙

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed プルリクエストのタイトルは、AndroidManifest.xmlからserviceを削除するという変更内容を正確に反映しており、主な変更を明確に説明しています。
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.


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

@TinyKitten TinyKitten merged commit ef16a0c into dev Jan 25, 2026
4 checks passed
@TinyKitten TinyKitten deleted the TinyKitten-patch-1 branch January 25, 2026 06:12
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.

1 participant