Skip to content

Upgrade react-native-webview before RN 0.88 bump #101443

Closed

Description

@MelvinBot

Parent issue

Part of #101427 — must land before the main React Native 0.88 / Expo SDK 58 bump PR.

Background

Expo SDK 58 bundles react-native-webview 14.0.1. main is on 13.16.0. WebView declares react-native: *, so it is not gated on the RN bump and can land on the current RN 0.86 / Expo SDK 57 stack.

The only breaking change in 14.0.0 is that Android now requires API 24 (Nougat) or newer, with the legacy native file-upload paths removed. android/build.gradle already sets minSdkVersion = 24, so no Android floor change is needed.

Scope

  • Bump react-native-webview 13.16.0 → 14.0.1 in App (and Mobile-Expensify if required for HybridApp parity)
  • Rebase or remove the three existing patches under patches/react-native-webview/, all pinned to 13.16.0 in their filenames:
    • react-native-webview+13.16.0+001+fix-dataDetectorTypes.patch
    • react-native-webview+13.16.0+002+certificate-pinning.patch
    • react-native-webview+13.16.0+003+certificate-pinning-android.patch
  • Confirm whether the removed Android file-upload compatibility code affects our attachment-upload flows inside a WebView

Target

  • react-native-webview@14.0.1 (Expo SDK 58 bundled version)

Acceptance criteria

  • App builds on iOS, Android, and web
  • HybridApp builds if Mobile-Expensify changes are needed
  • Certificate pinning still works on iOS and Android (both pinning patches verified, not just re-applied)
  • dataDetectorTypes behavior unchanged
  • WebView-hosted flows (help/docs pages, third-party auth and integration flows) still load and function
  • Patch audit documented: keep, update, or remove
  • No new JS console errors

Notes

  • Keep this PR focused — do not include the RN / Expo version bump here
Issue OwnerCurrent Issue Owner: @war-in

Activity

  1. self-assigned this
    on Sep 17, 2026
  2. melvin-bot commented on Sep 21, 2026

    @melvin-bot

    @war-in Whoops! This issue is 2 days overdue. Let's get this updated quick!

  3. melvin-bot commented on Sep 23, 2026

    @melvin-bot

    @war-in Huh... This is 4 days overdue. Who can take care of this?

  4. melvin-bot commented on Sep 25, 2026

    @melvin-bot

    @war-in 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

  5. melvin-bot commented on Sep 29, 2026

    @melvin-bot

    @war-in 10 days overdue. Is anyone even seeing these? Hello?

  6. melvin-bot commented on Oct 1, 2026

    @melvin-bot

    @war-in 12 days overdue. Walking. Toward. The. Light...

  7. melvin-bot commented on Oct 5, 2026

    @melvin-bot

    This issue has not been updated in over 14 days. @Julesssss, @mountiny, @roryabraham, @shubham1206agra, @war-in, @jakubstec, @collectioneur eroding to Weekly issue.

  8. collectioneur commented on Oct 7, 2026

    @collectioneur
    Contributor

    We can close this since we didn’t need this for the react-native bump. Also, it was already bumped to 16.0.0 in another PR (#101069)
    cc: @mountiny

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

EngineeringInternalRequires API changes or must be handled by Expensify staffTaskWeeklyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions