From e3272810a500c373cecad8ea8a8094f7f37e5189 Mon Sep 17 00:00:00 2001 From: Irena Szukala Date: Tue, 16 Jun 2026 16:26:30 +0200 Subject: [PATCH 1/3] docs: remove stale README sections and sample permission - Remove the ProGuard section: the published AAR already bundles consumer ProGuard rules with the same keep rules, so manual setup is redundant. - Update Transitive Dependencies example from Glide to Coil, matching the SDK's current image-loading dependency. - Remove READ_EXTERNAL_STORAGE guidance: deprecated/ignored for media since Android 13, and the SDK no longer declares it. - Drop READ_EXTERNAL_STORAGE from the sample app manifest for consistency. Co-Authored-By: Claude Opus 4.8 (1M context) --- README.md | 21 ++------------------- sample/app/src/main/AndroidManifest.xml | 1 - 2 files changed, 2 insertions(+), 20 deletions(-) diff --git a/README.md b/README.md index 646e3cb..62923ed 100644 --- a/README.md +++ b/README.md @@ -95,17 +95,6 @@ A project with some basic example integrations is provided [here](https://github * Read our guide on Push Notifications for [FCM](https://developers.intercom.com/installing-intercom/docs/android-fcm-push-notifications). * Please contact us on [Intercom](https://www.intercom.com/) with any questions you may have, we're only a message away! -## ProGuard - -If you are using ProGuard, add the following rules: - -``` --keep class io.intercom.android.** { *; } --keep class com.intercom.** { *; } -``` - -You might also need to add rules for OkHttp, Okio and Retrofit which are dependencies used in this library. - ## Cordova/Phonegap Support Looking for Cordova/Phonegap support? We have a [Cordova Plugin](https://github.com/intercom/intercom-cordova) for Intercom 🎉 @@ -117,12 +106,6 @@ We include the [INTERNET](http://developer.android.com/reference/android/Manifes ``` -You will need to include the [READ\_EXTERNAL\_STORAGE](http://developer.android.com/reference/android/Manifest.permission.html#READ_EXTERNAL_STORAGE) permission if you have enabled attachments: - -```xml - -``` - You can also include [VIBRATE](http://developer.android.com/reference/android/Manifest.permission.html#VIBRATE) to enable vibration in push notifications: ```xml @@ -131,6 +114,6 @@ You can also include [VIBRATE](http://developer.android.com/reference/android/Ma ### Transitive Dependencies -Intercom Android SDK transitively depends on the above libraries. If your app is using any one of these libraries, they should at least be on the same major version that Intercom SDK is using. +Intercom Android SDK transitively depends on a number of common libraries (for example, Coil for image loading). If your app is using any one of these libraries, they should at least be on the same major version that the Intercom SDK is using. When there are two versions of a library at build time, Gradle automatically picks the newer version. -This means if you are currently using say Glide 3.x, your app would automatically get Glide 4.x after including Intercom. +This means if you are currently using say Coil 2.x and Intercom depends on a newer 2.x release, your app would automatically get the newer version after including Intercom. diff --git a/sample/app/src/main/AndroidManifest.xml b/sample/app/src/main/AndroidManifest.xml index 0bc35d4..1aeef08 100644 --- a/sample/app/src/main/AndroidManifest.xml +++ b/sample/app/src/main/AndroidManifest.xml @@ -2,7 +2,6 @@ - Date: Tue, 16 Jun 2026 16:29:06 +0200 Subject: [PATCH 2/3] docs: clarify Coil version example wording Co-Authored-By: Claude Opus 4.8 (1M context) --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 62923ed..f2e1c5b 100644 --- a/README.md +++ b/README.md @@ -116,4 +116,4 @@ You can also include [VIBRATE](http://developer.android.com/reference/android/Ma Intercom Android SDK transitively depends on a number of common libraries (for example, Coil for image loading). If your app is using any one of these libraries, they should at least be on the same major version that the Intercom SDK is using. When there are two versions of a library at build time, Gradle automatically picks the newer version. -This means if you are currently using say Coil 2.x and Intercom depends on a newer 2.x release, your app would automatically get the newer version after including Intercom. +This means if you are currently using say Coil 2.4 and Intercom depends on a newer 2.x release, your app would automatically get the newer version after including Intercom. From 36068baaac46312ca6fcb73ce6b8598de52b4334 Mon Sep 17 00:00:00 2001 From: Irena Szukala Date: Tue, 16 Jun 2026 16:30:08 +0200 Subject: [PATCH 3/3] docs: use 1.x vs 2.x example for Coil version bump Co-Authored-By: Claude Opus 4.8 (1M context) --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index f2e1c5b..71309d3 100644 --- a/README.md +++ b/README.md @@ -116,4 +116,4 @@ You can also include [VIBRATE](http://developer.android.com/reference/android/Ma Intercom Android SDK transitively depends on a number of common libraries (for example, Coil for image loading). If your app is using any one of these libraries, they should at least be on the same major version that the Intercom SDK is using. When there are two versions of a library at build time, Gradle automatically picks the newer version. -This means if you are currently using say Coil 2.4 and Intercom depends on a newer 2.x release, your app would automatically get the newer version after including Intercom. +This means if you are currently using say Coil 1.x, your app would automatically get Coil 2.x after including Intercom.