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

Upgrade to RN 0.72 #572

Merged
merged 44 commits into from
Nov 27, 2023
Merged

Upgrade to RN 0.72 #572

merged 44 commits into from
Nov 27, 2023

Conversation

nazli-stripe
Copy link
Collaborator

@nazli-stripe nazli-stripe commented Nov 16, 2023

Summary

Upgrades RN version used in the project to latest 0.71 and Android Gradle Plugin to 8.1.1. These changes allowed us the necessary setup to bump the native SDKs to 3.2.0, which are also done in this PR.

Motivation

SDK update

Testing

  • I tested this manually
  • I added automated tests

Documentation

Select one:

  • I have added relevant documentation for my changes.
  • This PR does not result in any developer-facing changes.

android/build.gradle Outdated Show resolved Hide resolved
@nazli-stripe nazli-stripe marked this pull request as ready for review November 22, 2023 21:45
bitrise.yml Show resolved Hide resolved
dev-app/android/app/build.gradle Outdated Show resolved Hide resolved
Comment on lines +119 to +120
// The version of react-native is set by the React Native Gradle Plugin
implementation("com.facebook.react:react-android")
Copy link
Contributor

Choose a reason for hiding this comment

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

Sweet

Comment on lines 1 to 6
/**
* Copyright (c) Meta Platforms, Inc. and affiliates.
*
* <p>This source code is licensed under the MIT license found in the LICENSE file in the root
* directory of this source tree.
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably remove this, I think

Copy link
Collaborator

@bric-stripe bric-stripe left a comment

Choose a reason for hiding this comment

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

ios pieces lgtm

@nazli-stripe nazli-stripe merged commit ce3cb02 into main Nov 27, 2023
1 check passed
@nazli-stripe nazli-stripe deleted the nazli/rn-update branch September 7, 2024 17:13
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.

3 participants