Skip to content

Conversation

floydkim
Copy link
Member

@floydkim floydkim commented Mar 2, 2025

resolves: #13

Android

It uses the reload method of ReactDelegate introduced in React Native 0.74.
For versions below 0.74, it behaves the same as the previous implementation.

iOS

It uses RCTTriggerReloadCommandListeners instead of [_bridge reload] (reference)

Test Plan

  • download update
  • reload to install update
  • clear update
  • get update metadata

Test environments

  • iOS / Android x Debug / Release x Old Architecture / New Architecture / New Architecture Bridgeless

Test Targets (the latest version of each minor version)

RN Version Old Architecture New Architecture New Architecture Bridgeless
0.73.11 Unsupported
0.74.7
0.75.5
0.76.7
0.77.1
0.78.0

@floydkim floydkim changed the base branch from master to v10 March 9, 2025 08:46
@floydkim floydkim changed the title fix(Native): resolve reload issue on Fabric+Bridgeless fix(Native): Support New Architecture (with Bridgeless mode) Mar 9, 2025
@floydkim floydkim force-pushed the fabric-bridgeless branch from 1e44551 to 29e8e7d Compare March 9, 2025 08:53
…geless release)

I found an issue in the Android RN 0.74.7 (New Architecture + Bridgeless) release build where the CodePush instance is newly created when reloading or updating downloads, causing the JS bundle file name to be reset to null.

Since the CodePush instance is not newly created in Old Architecture and New Architecture Bridge mode, I modified it to be a singleton in Bridgeless mode as well.
@floydkim floydkim changed the title fix(Native): Support New Architecture (with Bridgeless mode) fix(Native): Support New Architecture (Bridge/Bridgeless) Mar 9, 2025
@floydkim floydkim changed the title fix(Native): Support New Architecture (Bridge/Bridgeless) feat(Native): Support New Architecture (Bridge/Bridgeless) Mar 16, 2025
@floydkim floydkim merged commit 584f8a4 into v10 Mar 16, 2025
1 check passed
@floydkim floydkim deleted the fabric-bridgeless branch March 16, 2025 13:43
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.

1 participant