HomePhabricator
Diffusion Comm 7d56562e81b9

[native] Fix Babel preset patch after React Native 0.70.15 upgrade

Description

[native] Fix Babel preset patch after React Native 0.70.15 upgrade

Summary:
This addresses ENG-9678, which tracks a regression caused by D13700.

Before that diff, we had just one version of metro-react-native-babel-transformer shared by Expo and React Native. After the diff, we have two versions:

  • The old metro-react-native-babel-transformer@0.72.3, used by Expo, and patched here.
  • The new metro-react-native-babel-transformer@0.72.4. It's in two places: as a transitive dependency of metro and a transitive dependency of metro-react-native-babel-transformer.

Because we now have two versions, the old patch only applies to one version, and the issue addressed by the patch was reintroduced.

I did some testing and found that of the three locations, we need two of them for the issue to be addressed: the old location, and one of the new locations, which is patched here. For some reason, the transitive dependency of metro itself does not appear to need to be patched.

Test Plan: I compiled release builds to my iOS simulator and physical Android device. I logged in as my "ashoat" user to production and confirmed I was able to resolve both normal "onchain" ENS names like "ashoat.eth" as well as "offchain" ENS names like "ashoat.base.eth".

Reviewers: will, varun

Reviewed By: will

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D13748

Details

Provenance
ashoatAuthored on Thu, Oct 17, 12:44 PM
Reviewer
will
Differential Revision
D13748: [native] Fix Babel preset patch after React Native 0.70.15 upgrade
Parents
rCOMMfbaa1e172e5f: [lib] Use setLocalFID when updating local FID to match identity
Branches
Unknown
Tags
Unknown