Skip to content

Upgrade Expo to fix Android Hybrid app crash #59377

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.

Sign up for GitHub

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

Merged
mountiny merged 5 commits into from
Apr 4, 2025
Merged

Upgrade Expo to fix Android Hybrid app crash #59377

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b7544f4
Upgrade expo to 52.0.41
VickyStash Mar 31, 2025
6b340b1
Podfile.lock updates
VickyStash Apr 1, 2025
ec58f21
Merge branch 'refs/heads/main' into VickyStash/bugfix/59335-fix-crash…
VickyStash Apr 1, 2025
a84e41c
package-lock.json updates
VickyStash Apr 1, 2025
fec435f
gh actions updates
VickyStash Apr 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
408 changes: 355 additions & 53 deletions .github/actions/javascript/checkAndroidStatus/index.js
View file
Open in desktop

Large diffs are not rendered by default.

408 changes: 355 additions & 53 deletions .github/actions/javascript/getAndroidRolloutPercentage/index.js
View file
Open in desktop

Large diffs are not rendered by default.

16 changes: 8 additions & 8 deletions ios/Podfile.lock
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,11 @@ PODS:
- ReactCommon/turbomodule/bridging
- ReactCommon/turbomodule/core
- Yoga
- Expo (52.0.28):
- Expo (52.0.41):
- ExpoModulesCore
- ExpoAsset (11.0.2):
- ExpoAsset (11.0.5):
- ExpoModulesCore
- ExpoFont (13.0.3):
- ExpoFont (13.0.4):
- ExpoModulesCore
- ExpoImage (2.0.4):
- ExpoModulesCore
Expand All @@ -72,7 +72,7 @@ PODS:
- EXImageLoader
- ExpoModulesCore
- SDWebImageWebPCoder
- ExpoModulesCore (2.2.0):
- ExpoModulesCore (2.2.3):
- DoubleConversion
- glog
- hermes-engine
Expand Down Expand Up @@ -3318,12 +3318,12 @@ SPEC CHECKSUMS:
EXAV: 57ea461614a714b8b8fcc72c6fbc298b8f1ec78b
EXImageLoader: 759063a65ab016b836f73972d3bb25404888713d
expensify-react-native-background-task: f180077e77a41952128fdc2c83ce1d980994d125
Expo: c124b1c2129e79a181c16c364aec9add939bea24
ExpoAsset: 6cc988c426b5fc997859cc0f8f6728d1376bf53f
ExpoFont: 8b14db1afebf62e9774613a082e402985f4e747c
Expo: 3a8a619381412bbf8537746ec27b485be01c8a1f
ExpoAsset: 0687fe05f5d051c4a34dd1f9440bd00858413cfe
ExpoFont: 773955186469acc5108ff569712a2d243857475f
ExpoImage: 3099001359e4414d60addd7c3e00a5d949df41e0
ExpoImageManipulator: 43c7bb3ecccbe993054d2e9131c8dcbe54f1385b
ExpoModulesCore: 602c3b9247a11fd714e37c9c7452cac130b9715f
ExpoModulesCore: 99d5de91c92f514765d101d1e4988a5f387995e8
fast_float: 06eeec4fe712a76acc9376682e4808b05ce978b6
FBLazyVector: 79c4b7ec726447eec5f8593379466bd9fde1aa14
Firebase: 629510f1a9ddb235f3a7c5c8ceb23ba887f0f814
Expand Down
457 changes: 262 additions & 195 deletions package-lock.json
View file
Open in desktop

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,10 +123,10 @@
"dom-serializer": "^0.2.2",
"domhandler": "^4.3.0",
"expensify-common": "2.0.127",
"expo": "52.0.28",
"expo-asset": "^11.0.2",
"expo": "52.0.41",
"expo-asset": "^11.0.5",
"expo-av": "^15.0.2",
"expo-font": "^13.0.3",
"expo-font": "^13.0.4",
"expo-image": "^2.0.4",
"expo-image-manipulator": "^13.0.6",
"fast-equals": "^4.0.3",
Expand Down
47 changes: 0 additions & 47 deletions patches/expo+52.0.28+001+dont-change-formdata.patch
View file
Open in desktop

@war-in war-in Apr 1, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is that included in the new version? Just making sure it's not removed by mistake

@VickyStash VickyStash Apr 1, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yep, necessary updates were merged to the expo around one month ago: https://github.com/expo/expo/pull/35243/files

This file was deleted.

0 patches/expo-asset+11.0.2+002+initial.patch → patches/expo-asset+11.0.5+002+initial.patch
View file
Open in desktop
File renamed without changes.
0 ...core+2.2.0+001+disableViewRecycling.patch → ...core+2.2.3+001+disableViewRecycling.patch
View file
Open in desktop
File renamed without changes.
0 ...s-core+2.2.0+002+change-global-type.patch → ...s-core+2.2.3+002+change-global-type.patch
View file
Open in desktop
File renamed without changes.