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

task(SDK-3620) - Upgrades AGP to 8.2.1 #545

Merged
merged 15 commits into from
Feb 16, 2024

Conversation

Anush-Shand
Copy link
Contributor

https://wizrocket.atlassian.net/browse/SDK-3620

Upgrades to AGP 8 and addresses build issues because of missing classes when minify is enabled

@Anush-Shand
Copy link
Contributor Author

Anush-Shand commented Jan 29, 2024

The missing classes error for exoplayer and glide has been addressed as they pertain to optional dependencies in the clevertap-sdk.
For Huawei and Baidu these errors arise from their dependency. Their docs respectively specify adding the produard -dontwarn rules
Do we wish to add rules for these 3rd party libraries to our consumer-rules itself or leave this to the client. Adding it to our consumer-rules may mask some other issue for the client

Screenshot 2024-01-29 at 1 15 03 PM Screenshot 2024-01-29 at 1 15 33 PM

@@ -142,7 +142,7 @@ dependencies {
remoteImplementation("com.clevertap.android:clevertap-geofence-sdk:1.2.0")
remoteImplementation("com.clevertap.android:clevertap-xiaomi-sdk:1.5.2")
remoteImplementation("com.clevertap.android:push-templates:1.1.0")
remoteImplementation("com.clevertap.android:clevertap-hms-sdk:1.3.2")
remoteImplementation("com.clevertap.android:clevertap-hms-sdk:1.3.3")

stagingImplementation("com.clevertap.android:clevertap-android-sdk:5.2.1")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upgrade this as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yess. All the sample app changes can be updated in a separated pr? I shall revert this for now

@Anush-Shand Anush-Shand merged commit 25243b4 into develop Feb 16, 2024
4 checks passed
@Anush-Shand Anush-Shand deleted the task/SDK-3620/upgrade_agp_8 branch February 16, 2024 06:58
@Anush-Shand Anush-Shand restored the task/SDK-3620/upgrade_agp_8 branch February 16, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants