Skip to content

Upgrade playground react-native's version to 0.66 #7034

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

Closed
wants to merge 54 commits into from
Closed

Conversation

yogevbd
Copy link
Collaborator

@yogevbd yogevbd commented Mar 15, 2021

Release Notes

ReactNative

  • ReactNative 0.66
  • Fix Warnings and deprecations.

Android

  • Android Gradle Plugin 7.
  • Target API Raised to API 30.
  • Reanimated 2 (still in beta, playground SET fixes needed).
  • Hermes enabled by default.
  • Using JAVA 11 for IDE.
  • Fix Warnings and deprecations.
  • Removing Flavors.
  • Kotling 1.5.x.

iOS

@yogevbd yogevbd added this to the 8.0.0 milestone Mar 15, 2021
@swabbass
Copy link
Contributor

swabbass commented Apr 11, 2021

I Think also in this upgrade we can bump a major, and drop support for old ReactNative versions in android (Flavors), we might support RN 0.63+ version at least in the android side since it's taking a lot of resources maintaining these backwards compatibility behaviours across RN versions.

@guyca wdyt?

@stale
Copy link

stale bot commented May 27, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the 🏚 stale label May 27, 2021
@yogevbd
Copy link
Collaborator Author

yogevbd commented May 27, 2021

@swabbass we will drop support for RN versions lower than 0.64 in V8 🔥

@birkir
Copy link
Contributor

birkir commented May 29, 2021

It would make sense to use hermes on android too, right?

@swabbass
Copy link
Contributor

swabbass commented May 30, 2021

It would make sense to use Hermes on android too, right?

Hey @birkir :]
The js side of RNN works with Hermes and JSC, it's up to the host app to use Hermes or other variants, we can enable it by default, but it feels like there is no added value to the lib by enabling it in the Playground module of RNN.

@birkir
Copy link
Contributor

birkir commented May 30, 2021

It would make sense to use Hermes on android too, right?

Hey @birkir :]

The js side of RNN works with Hermes and JSC, it's up to the host app to use Hermes or other variants, we can enable it by default, but it feels like there is no added value to the lib by enabling it in the Playground module of RNN.

Hey, you probably misunderstood me.

This PR enables hermes on iOS, but not Android and I was only commenting on the inconsistency in the Playground app.

I know the package has nothing to do with Hermes outwards.

@swabbass swabbass mentioned this pull request May 31, 2021
@swabbass
Copy link
Contributor

swabbass commented Jun 1, 2021

It would make sense to use Hermes on android too, right?

Hey @birkir :]
The js side of RNN works with Hermes and JSC, it's up to the host app to use Hermes or other variants, we can enable it by default, but it feels like there is no added value to the lib by enabling it in the Playground module of RNN.

Hey, you probably misunderstood me.

This PR enables hermes on iOS, but not Android and I was only commenting on the inconsistency in the Playground app.

I know the package has nothing to do with Hermes outwards.

Hey sorry, I did not get that from your comment but hey, you have a point here, sure.

# Conflicts:
#	lib/android/app/src/main/java/com/reactnativenavigation/options/HardwareBackButtonOptions.kt
#	lib/android/app/src/main/java/com/reactnativenavigation/viewcontrollers/modal/ModalPresenter.java
#	lib/android/app/src/main/java/com/reactnativenavigation/viewcontrollers/stack/StackController.java
#	lib/android/app/src/reactNative63/java/com/reactnativenavigation/react/NavigationReactNativeHost.java
# Conflicts:
#	lib/android/app/src/main/java/com/reactnativenavigation/options/HardwareBackButtonOptions.kt
#	lib/android/app/src/main/java/com/reactnativenavigation/viewcontrollers/modal/ModalPresenter.java
#	lib/android/app/src/main/java/com/reactnativenavigation/viewcontrollers/stack/StackController.java
#	lib/android/app/src/reactNative63/java/com/reactnativenavigation/react/NavigationReactNativeHost.java
@svbutko
Copy link
Contributor

svbutko commented Oct 5, 2021

What about dropping support for iOS 11 and iOS 12?
And keeping last 3 major versions (13,14,15)?
wdyt?

It will remove good part of the code and separated tests

@svbutko
Copy link
Contributor

svbutko commented Oct 5, 2021

https://developer.apple.com/support/app-store/

With iOS 15 release these numbers will go even lower

@yogevbd
Copy link
Collaborator Author

yogevbd commented Oct 5, 2021

@svbutko I need to check with our production team if they plan on removing support for iOS 12 in the Wix app. If so we defiantly can

@svbutko
Copy link
Contributor

svbutko commented Oct 7, 2021

There are several requirements to upgrade AGP properly to 7.0.2:
facebook/react-native#32186

@svbutko
Copy link
Contributor

svbutko commented Oct 7, 2021

And if it's going to be AGP 7.0.2 then I would mention how to install JDK 11:

brew install --cask adoptopenjdk/openjdk/adoptopenjdk11

Since most of the users have JDK 8 only installed

@swabbass
Copy link
Contributor

swabbass commented Oct 7, 2021

And if it's going to be AGP 7.0.2 then I would mention how to install JDK 11:

brew install --cask adoptopenjdk/openjdk/adoptopenjdk11

Since most of the users have JDK 8 only installed

yup, we can mention that it is already running e2e locally in a release.
Our CI should have JDK 11 installed also.

@swabbass
Copy link
Contributor

swabbass commented Oct 7, 2021

Users can choose to have such gradle.properties:

# For adoptopenjdk: brew install --cask adoptopenjdk/openjdk/adoptopenjdk11
#org.gradle.java.home=/Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home

# For Android Studio embedded JVM version
org.gradle.java.home=/Applications/Android Studio.app/Contents/jre/Contents/Home

@svbutko wdyt?

@svbutko
Copy link
Contributor

svbutko commented Oct 7, 2021

Users can choose to have such gradle.properties:

# For adoptopenjdk: brew install --cask adoptopenjdk/openjdk/adoptopenjdk11
#org.gradle.java.home=/Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home

# For Android Studio embedded JVM version
org.gradle.java.home=/Applications/Android Studio.app/Contents/jre/Contents/Home

@svbutko wdyt?

LGTM, but I think it's worth mentioning Windows users too:

choco install -y nodejs.install openjdk11

I myself on macOS but I know many people use Windows for development

@swabbass
Copy link
Contributor

swabbass commented Oct 7, 2021

Users can choose to have such gradle.properties:

# For adoptopenjdk: brew install --cask adoptopenjdk/openjdk/adoptopenjdk11
#org.gradle.java.home=/Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home

# For Android Studio embedded JVM version
org.gradle.java.home=/Applications/Android Studio.app/Contents/jre/Contents/Home

@svbutko wdyt?

LGTM, but I think it's worth mentioning Windows users too:

choco install -y nodejs.install openjdk11

I myself on macOS but I know many people use Windows for development

Showing love for Windows devs.

@svbutko
Copy link
Contributor

svbutko commented Oct 7, 2021

Users can choose to have such gradle.properties:

# For adoptopenjdk: brew install --cask adoptopenjdk/openjdk/adoptopenjdk11
#org.gradle.java.home=/Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home

# For Android Studio embedded JVM version
org.gradle.java.home=/Applications/Android Studio.app/Contents/jre/Contents/Home

@svbutko wdyt?

LGTM, but I think it's worth mentioning Windows users too:
choco install -y nodejs.install openjdk11
I myself on macOS but I know many people use Windows for development

Showing love for Windows devs.

I started working with RN around 5 years ago on Windows (transitioned to macOS in 2019) and I went through all of the possible issues so I know what it's like 😄

@swabbass
Copy link
Contributor

swabbass commented Oct 7, 2021

Update:

  • We got a couple of issues running unit tests (JS), babel related as it looks:
    Screen Shot 2021-10-07 at 10 19 45

  • Also, E2E we got jasmine stuff that seems to be broken:
    Screen Shot 2021-10-07 at 9 47 11

  • Android Build debug/release are passing.

@retyui
Copy link
Contributor

retyui commented Oct 9, 2021

How about to use microsoft/react-native-test-app to easily support playground app?

@swabbass
Copy link
Contributor

Closed in favour of #7305, this will be used as a reference for changes on native.

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.

Navigation event listeners are not working
7 participants