Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only depend on ads-identifier #41

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ dependencies {
implementation "com.bazaarvoice.bvandroidsdk:conversations-ui:$demoAppSharedVersions.bvSdkVersion"
}
//BVSDK Dependencies
implementation 'com.google.android.gms:play-services-ads:15.0.0'
implementation sdkDep.playServiceAdsIdentifier
implementation 'com.google.android.gms:play-services-maps:16.0.0'
implementation 'com.google.code.gson:gson:2.8.5'
implementation 'com.squareup.okhttp3:okhttp:3.12.0'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,6 @@
import android.graphics.Point;
import android.view.Display;

import com.google.android.gms.ads.AdRequest;

import org.json.JSONException;
import org.json.JSONObject;

Expand All @@ -17,22 +15,6 @@ public class DemoUtils {
public static final int MAX_IMAGE_WIDTH = 1440;
public static final int MAX_IMAGE_HEIGHT = 2560;

// private Picasso picasso;
// private Context context;
//
// public DemoUtils(Context context) {
// this.context = context;
// }

// public Picasso picassoThumbnailLoader() {
// if (picasso == null) {
// picasso = new Picasso.Builder(context)
// .defaultBitmapConfig(Bitmap.Config.RGB_565)
// .build();
// }
// return picasso;
// }

public static <ReturnType> ReturnType safeParse(String key, JSONObject jsonObject) {
return safeParse(key, jsonObject, null);
}
Expand All @@ -59,21 +41,6 @@ public static String safeParseString(String key, JSONObject jsonObject, String d
return value != null ? value : defaultValue;
}

public static String getDfpAdErrorMessage(int errorCode) {
switch (errorCode) {
case AdRequest.ERROR_CODE_INTERNAL_ERROR:
return "Something happened internally; for instance, an invalid response was received from the ad server.";
case AdRequest.ERROR_CODE_NETWORK_ERROR:
return "The ad request was unsuccessful due to network connectivity.";
case AdRequest.ERROR_CODE_INVALID_REQUEST:
return "The ad request was invalid; for instance, the ad unit ID was incorrect.";
case AdRequest.ERROR_CODE_NO_FILL:
return "The ad request was successful, but no ad was returned due to lack of ad inventory.";
default:
return "unknown error code";
}
}

public static Point getScreenDimensions(Activity activity) {
Display display = activity.getWindowManager().getDefaultDisplay();
Point size = new Point();
Expand Down
84 changes: 0 additions & 84 deletions app/src/main/res/layout/row_ad.xml

This file was deleted.

4 changes: 2 additions & 2 deletions bvanalytics/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ android {

dependencies {
implementation sdkDep.supportAnnotations
implementation sdkDep.playServiceAds
implementation sdkDep.playServiceAdsIdentifier
implementation sdkDep.okhttp3

// Dependencies for local unit tests
testImplementation sdkDep.playServiceAds
testImplementation sdkDep.playServiceAdsIdentifier
testImplementation sdkDep.gson
testImplementation sdkTestDep.junit
testImplementation sdkTestDep.mockito
Expand Down
2 changes: 1 addition & 1 deletion bvauthiovation/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ dependencies {
testCompileOnly project(':bvcommon')
testCompileOnly project(':bvanalytics')
testCompileOnly sdkDep.gson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
}

apply from: rootProject.file('gradle/setup-signing.gradle')
Expand Down
2 changes: 1 addition & 1 deletion bvcommon/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ dependencies {
implementation project(':bvanalytics')
implementation sdkDep.okhttp3
implementation sdkDep.recyclerview
implementation sdkDep.playServiceAds
implementation sdkDep.playServiceAdsIdentifier
implementation sdkDep.gson

// Dependencies for local unit tests
Expand Down
2 changes: 1 addition & 1 deletion bvconversations/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ dependencies {
testImplementation sdkTestDep.robolectric
testImplementation project(':bvanalytics')
testImplementation sdkTestDep.androidJson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
testCompileOnly sdkDep.gson
}

Expand Down
2 changes: 1 addition & 1 deletion bvconversationsui/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ dependencies {
testImplementation sdkTestDep.robolectric
testImplementation project(':bvanalytics')
testCompileOnly sdkTestDep.androidJson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
testCompileOnly sdkDep.gson
}

Expand Down
2 changes: 1 addition & 1 deletion bvcurations/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ dependencies {
testImplementation sdkTestDep.robolectric
testCompileOnly project(':bvcommon')
testCompileOnly project(':bvanalytics')
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
testCompileOnly sdkTestDep.androidJson
testCompileOnly sdkDep.gson
}
Expand Down
2 changes: 1 addition & 1 deletion bvcurationsui/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ dependencies {
testCompileOnly sdkTestDep.robolectric
testCompileOnly project(':bvanalytics')
testCompileOnly sdkDep.gson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
}

apply from: rootProject.file('gradle/setup-signing.gradle')
Expand Down
2 changes: 1 addition & 1 deletion bvlocation/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ dependencies {
testCompileOnly sdkTestDep.robolectric
testCompileOnly project(':bvanalytics')
testCompileOnly sdkDep.gson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
}

apply from: rootProject.file('gradle/setup-signing.gradle')
Expand Down
2 changes: 1 addition & 1 deletion bvnotifications/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ dependencies {
testImplementation sdkTestDep.robolectric
testCompileOnly project(':bvanalytics')
testCompileOnly sdkDep.gson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
}

apply from: rootProject.file('gradle/setup-signing.gradle')
Expand Down
4 changes: 2 additions & 2 deletions bvrecommendations/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ dependencies {
compileOnly sdkDep.recyclerview
compileOnly sdkDep.gson
compileOnly sdkDep.okhttp3
compileOnly sdkDep.playServiceAds
compileOnly sdkDep.playServiceAdsIdentifier

// Dependencies for local unit tests
testImplementation sdkTestDep.junit
Expand All @@ -15,7 +15,7 @@ dependencies {
testImplementation sdkTestDep.robolectric
testCompileOnly project(':bvanalytics')
testCompileOnly sdkDep.gson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
}

apply from: rootProject.file('gradle/setup-signing.gradle')
Expand Down
2 changes: 1 addition & 1 deletion bvstorenotifications/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ dependencies {
testCompileOnly project(':bvanalytics')
testCompileOnly project(':bvcommon')
testCompileOnly sdkDep.gson
testCompileOnly sdkDep.playServiceAds
testCompileOnly sdkDep.playServiceAdsIdentifier
}

apply from: rootProject.file('gradle/setup-signing.gradle')
Expand Down
10 changes: 5 additions & 5 deletions dependencies.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ ext {
okhttp3Version : '3.10.0',
supportLibraryVersion : '28.0.0',
gimbalVersion : '2.72',
playServicesVersion : '15.0.0',
playServicesAdsIdentifier : '17.1.0',
robolectric: '4.3',
robolectricAddOn : '4.3',
androidx: '1.0.0'
Expand All @@ -36,10 +36,10 @@ ext {
supportAnnotations : "androidx.annotation:annotation:$sdkSharedVersions.androidx",

// common
okhttp3 : "com.squareup.okhttp3:okhttp:$sdkSharedVersions.okhttp3Version",
gson : 'com.google.code.gson:gson:2.8.5',
playServiceAds : "com.google.android.gms:play-services-ads:$sdkSharedVersions.playServicesVersion",
recyclerview : "androidx.recyclerview:recyclerview:$sdkSharedVersions.androidx",
okhttp3 : "com.squareup.okhttp3:okhttp:$sdkSharedVersions.okhttp3Version",
gson : 'com.google.code.gson:gson:2.8.5',
playServiceAdsIdentifier : "com.google.android.gms:play-services-ads-identifier:$sdkSharedVersions.playServicesAdsIdentifier",
recyclerview : "androidx.recyclerview:recyclerview:$sdkSharedVersions.androidx",

// location
gimbal : "com.gimbal.android.v2:gimbal-sdk:$sdkSharedVersions.gimbalVersion",
Expand Down