NWACus / avy

Mobile-Native Viewing of NAC Avalanche Forecasts
MIT License
10 stars 6 forks source link

Upgrade expo to SDK 50 #674

Closed beaucollins closed 7 months ago

beaucollins commented 7 months ago

The @bothrs/expo-mixpanel-analytics dependency is keeping us pinned to an old @expo/config that expo doctor says needs to be updated.


npx expo install expo@50
npx expo install --fix
yarn add @expo/config@^8.5.0
yarn add @types/react@^18.2.4
yarn add typescript@^5.3.0

This upgrade expo and some dependences. However the builds then failed with an error referencing metro-react-native-babel-preset.

The fix for this was upgrading to a newer react-native-svg-transformer

yarn add  react-native-svg-tranformer@^1.3.0

Next problem was something about utils.makeFifoCache missing. The stack trace showed this coming from the sentry-expo package.

The SDK 50 migration instructions say that sentry-expo was merged into @sentry/react-native but should still work. That doesn't seem to be the case for us, so I migrated to using the latest @sentry/react-native.

Build now works on:

Android

image

iOS

image