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

Updated React Native, added flipper workarounds for iOS 14.5+ dev #846

Closed
wants to merge 5 commits into from

Conversation

jmarchi
Copy link
Contributor

@jmarchi jmarchi commented Jun 8, 2021

Description of change

Key changes:

Libraries
react-native to 0.64.2
react-native-webview to 11.6.2
react-test-renderer to 17.02
react to 17.02

Podfile
Forced Flipper versions to older versions to allow building to iOS 14.5+ (https://github.com/facebook/react-native/releases/tag/v0.64.1)
Added links to the OpenSSL-Universal framework used by Flipper to allow it to build on case sensitive file systems (facebook/react-native#28409 (comment))

Unit Tests
Updated path to NativeAnimatedHelper in test setup to work in RN 0.64.x
Mocked PanGestureHandler in react-native-gesture-handler to return a regular view to allow unit tests to run

Original issue(s)

department-of-veterans-affairs/va.gov-team#21657

@jmarchi jmarchi changed the title Updated React Native, added pod workarounds for iOS 14.5+ dev Updated React Native, added flipper workarounds for iOS 14.5+ dev Jun 8, 2021
@@ -19,8 +19,13 @@ target 'VAMobile' do
#
# Note that if you have use_frameworks! enabled, Flipper will not work and
# you should disable these next few lines.
use_flipper!
use_flipper!('Flipper' => '0.75.1', 'Flipper-Folly' => '2.5.3', 'Flipper-RSocket' => '1.3.1')
Copy link
Contributor

Choose a reason for hiding this comment

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

do we want to go to 0.87.0 ?

@jmarchi jmarchi closed this Jun 9, 2021
@jmarchi jmarchi deleted the jm.updates branch June 9, 2021 17:06
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.

2 participants