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

@W-13557285: [Android] Upgrade to Gradle 8.x and AGP 8.x Via RN 0.72.x #345

Conversation

JohnsonEricAtSalesforce
Copy link
Contributor

@JohnsonEricAtSalesforce JohnsonEricAtSalesforce commented Nov 16, 2023

🎸 Ready For Final Review 🥁

Be aware this pull request is one of four related to the React Native 0.72.7, Gradle 8.2 and Android Gradle Plugin 8.2.0 upgrades.
👉🏻 forcedotcom/SalesforceMobileSDK-Android#2492
👉🏻 forcedotcom/SalesforceMobileSDK-CordovaPlugin#631
👉🏻 #345
👉🏻 forcedotcom/SalesforceMobileSDK-Templates#387

This prepares SalesforceMobileSDK-ReactNative for supporting React Native 0.72.7, Gradle 8.2, Android Gradle Plugin 8.2.0 and Android 34 in the other three repositories listed above. Overall, the changes here are relatively small though the platform update has significant impacts to downstream projects.

I tested this primarily by using forcereact to generate every React Native template I could conceive of. Everything compiled and ran OK for both Android and iOS.

@wmathurin
Copy link
Contributor

Also could you subject of the PR reflect the fact it has a RN upgrade?

@JohnsonEricAtSalesforce JohnsonEricAtSalesforce force-pushed the feature/w-13557285-gradle-8-react-native-0.72.6 branch from cec3c9c to 4f129dc Compare November 16, 2023 19:10
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce changed the title @W-13557285: [Android] Upgrade to Gradle 8.x and AGP 8.x @W-13557285: [Android] Upgrade to Gradle 8.x and AGP 8.x Via RN 0.72.x Nov 16, 2023
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce force-pushed the feature/w-13557285-gradle-8-react-native-0.72.6 branch from 4f129dc to 55b9b3e Compare November 29, 2023 23:37
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce force-pushed the feature/w-13557285-gradle-8-react-native-0.72.6 branch from 55b9b3e to eacd3c4 Compare November 30, 2023 16:33
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce force-pushed the feature/w-13557285-gradle-8-react-native-0.72.6 branch from eacd3c4 to 6f9f691 Compare November 30, 2023 17:20
iosTests/package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
Copy link
Contributor

@brandonpage brandonpage left a comment

Choose a reason for hiding this comment

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

LGTM, just remember to switch JohnsonEricAtSalesforce repo url back to forcedotcom.

@JohnsonEricAtSalesforce JohnsonEricAtSalesforce marked this pull request as ready for review December 13, 2023 21:18
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce force-pushed the feature/w-13557285-gradle-8-react-native-0.72.6 branch from 5a47110 to 3e5c65c Compare December 14, 2023 01:25
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce merged commit 0515abc into forcedotcom:dev Dec 15, 2023
3 of 5 checks passed
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce deleted the feature/w-13557285-gradle-8-react-native-0.72.6 branch December 15, 2023 20:05
@wmathurin wmathurin mentioned this pull request Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants