Skip to content

Commit

Permalink
task(notification) - remove unused notification icon url from core an…
Browse files Browse the repository at this point in the history
…d push templates SDK-3772 (#600)

Co-authored-by: CTLalit <[email protected]>
  • Loading branch information
piyush-kukadiya and CTLalit authored Jul 8, 2024
1 parent ae46481 commit 4870960
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ public interface Constants {
int DEFINE_VARS_EVENT = 8;
String variablePayloadType = "varsPayload";
String WZRK_FETCH = "wzrk_fetch";
String ICON_BASE_URL = "http://static.wizrocket.com/android/ico/";
String NOTIFICATION_CLICKED_EVENT_NAME = "Notification Clicked";
String NOTIFICATION_VIEWED_EVENT_NAME = "Notification Viewed";
String SC_OUTGOING_EVENT_NAME = "SCOutgoing";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,6 @@ class NotificationBitmapDownloadRequestHandler(
)
}

// Simply stream the bitmap
if (!srcUrl.startsWith("http")) {
bitmapDownloadRequest.bitmapPath = "${Constants.ICON_BASE_URL}/$srcUrl"
}


val downloadedBitmap: DownloadedBitmap = iBitmapDownloadRequestHandler.handleRequest(bitmapDownloadRequest)

return Utils.getDownloadedBitmapPostFallbackIconCheck(fallbackToAppIcon, context, downloadedBitmap)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,7 @@ public static Bitmap getNotificationBitmap(String icoPath, boolean fallbackToApp
if (icoPath == null || icoPath.equals("")) {
return fallbackToAppIcon ? getAppIcon(context) : null;
}
// Simply stream the bitmap
if (!icoPath.startsWith("http")) {
icoPath = Constants.ICON_BASE_URL + "/" + icoPath;
}

Bitmap ic = getBitmapFromURL(icoPath,context);
return (ic != null) ? ic : ((fallbackToAppIcon) ? getAppIcon(context) : null);
}
Expand Down
8 changes: 4 additions & 4 deletions sample/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@ android {
applicationId "com.clevertap.demo"
minSdkVersion 21
targetSdkVersion 34
versionCode 200015
versionName "2.0.0-full"
versionCode 6020200
versionName "6.2.2"
multiDexEnabled true
testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
}
Expand Down Expand Up @@ -152,13 +152,13 @@ dependencies {
implementation "org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.72"
implementation "org.sonarsource.scanner.gradle:sonarqube-gradle-plugin:3.1.1"*/

remoteImplementation("com.clevertap.android:clevertap-android-sdk:6.2.0")
remoteImplementation("com.clevertap.android:clevertap-android-sdk:6.2.1")
remoteImplementation("com.clevertap.android:clevertap-geofence-sdk:1.3.0")
//remoteImplementation("com.clevertap.android:clevertap-xiaomi-sdk:1.5.4")
remoteImplementation("com.clevertap.android:push-templates:1.2.3")
remoteImplementation("com.clevertap.android:clevertap-hms-sdk:1.3.4")

stagingImplementation("com.clevertap.android:clevertap-android-sdk:6.2.0")
stagingImplementation("com.clevertap.android:clevertap-android-sdk:6.2.1")
stagingImplementation("com.clevertap.android:clevertap-geofence-sdk:1.3.0")
//stagingImplementation("com.clevertap.android:clevertap-xiaomi-sdk:1.5.4")
stagingImplementation("com.clevertap.android:push-templates:1.2.3")
Expand Down

0 comments on commit 4870960

Please sign in to comment.