Revert "Merge pull request #42592 from software-mansion-labs/kicu/con…

…current-react-lets-go" This reverts commit d047467, reversing changes made to 04f6598.
Expensify · Jul 3, 2024 · a3dd23a · a3dd23a
1 parent a9b4fef
commit a3dd23a
Show file tree

Hide file tree

Showing 5 changed files with 42 additions and 128 deletions.
diff --git a/patches/@react-navigation+core+6.4.11+001+fix-react-strictmode.patch b/patches/@react-navigation+core+6.4.11+001+fix-react-strictmode.patch
diff --git a/patches/react-native-reanimated+3.8.1+003+fix-strict-mode.patch b/patches/react-native-reanimated+3.8.1+003+fix-strict-mode.patch
diff --git a/src/App.tsx b/src/App.tsx
@@ -31,7 +31,6 @@ import {CurrentReportIDContextProvider} from './components/withCurrentReportID';
 import {EnvironmentProvider} from './components/withEnvironment';
 import {KeyboardStateProvider} from './components/withKeyboardState';
 import {WindowDimensionsProvider} from './components/withWindowDimensions';
-import CONFIG from './CONFIG';
 import Expensify from './Expensify';
 import useDefaultDragAndDrop from './hooks/useDefaultDragAndDrop';
 import {ReportIDsContextProvider} from './hooks/useReportIDs';
@@ -53,56 +52,51 @@ LogBox.ignoreLogs([
 
 const fill = {flex: 1};
 
-const StrictModeWrapper = CONFIG.USE_REACT_STRICT_MODE ? React.StrictMode : ({children}: {children: React.ReactElement}) => children;
-
 function App({url}: AppProps) {
     useDefaultDragAndDrop();
     OnyxUpdateManager();
-
     return (
-        <StrictModeWrapper>
-            <InitialURLContextProvider url={url}>
-                <GestureHandlerRootView style={fill}>
-                    <ComposeProviders
-                        components={[
-                            OnyxProvider,
-                            ThemeProvider,
-                            ThemeStylesProvider,
-                            ThemeIllustrationsProvider,
-                            SafeAreaProvider,
-                            PortalProvider,
-                            SafeArea,
-                            LocaleContextProvider,
-                            HTMLEngineProvider,
-                            WindowDimensionsProvider,
-                            KeyboardStateProvider,
-                            PopoverContextProvider,
-                            CurrentReportIDContextProvider,
-                            ScrollOffsetContextProvider,
-                            ReportAttachmentsProvider,
-                            PickerStateProvider,
-                            EnvironmentProvider,
-                            CustomStatusBarAndBackgroundContextProvider,
-                            ActiveElementRoleProvider,
-                            ActiveWorkspaceContextProvider,
-                            ReportIDsContextProvider,
-                            PlaybackContextProvider,
-                            FullScreenContextProvider,
-                            VolumeContextProvider,
-                            VideoPopoverMenuContextProvider,
-                            KeyboardProvider,
-                        ]}
-                    >
-                        <CustomStatusBarAndBackground />
-                        <ErrorBoundary errorMessage="NewExpensify crash caught by error boundary">
-                            <ColorSchemeWrapper>
-                                <Expensify />
-                            </ColorSchemeWrapper>
-                        </ErrorBoundary>
-                    </ComposeProviders>
-                </GestureHandlerRootView>
-            </InitialURLContextProvider>
-        </StrictModeWrapper>
+        <InitialURLContextProvider url={url}>
+            <GestureHandlerRootView style={fill}>
+                <ComposeProviders
+                    components={[
+                        OnyxProvider,
+                        ThemeProvider,
+                        ThemeStylesProvider,
+                        ThemeIllustrationsProvider,
+                        SafeAreaProvider,
+                        PortalProvider,
+                        SafeArea,
+                        LocaleContextProvider,
+                        HTMLEngineProvider,
+                        WindowDimensionsProvider,
+                        KeyboardStateProvider,
+                        PopoverContextProvider,
+                        CurrentReportIDContextProvider,
+                        ScrollOffsetContextProvider,
+                        ReportAttachmentsProvider,
+                        PickerStateProvider,
+                        EnvironmentProvider,
+                        CustomStatusBarAndBackgroundContextProvider,
+                        ActiveElementRoleProvider,
+                        ActiveWorkspaceContextProvider,
+                        ReportIDsContextProvider,
+                        PlaybackContextProvider,
+                        FullScreenContextProvider,
+                        VolumeContextProvider,
+                        VideoPopoverMenuContextProvider,
+                        KeyboardProvider,
+                    ]}
+                >
+                    <CustomStatusBarAndBackground />
+                    <ErrorBoundary errorMessage="NewExpensify crash caught by error boundary">
+                        <ColorSchemeWrapper>
+                            <Expensify />
+                        </ColorSchemeWrapper>
+                    </ErrorBoundary>
+                </ComposeProviders>
+            </GestureHandlerRootView>
+        </InitialURLContextProvider>
     );
 }
 

diff --git a/src/CONFIG.ts b/src/CONFIG.ts
@@ -96,5 +96,4 @@ export default {
         IOS_CLIENT_ID: '921154746561-s3uqn2oe4m85tufi6mqflbfbuajrm2i3.apps.googleusercontent.com',
     },
     GCP_GEOLOCATION_API_KEY: googleGeolocationAPIKey,
-    USE_REACT_STRICT_MODE: true,
 } as const;
diff --git a/src/setup/platformSetup/index.website.ts b/src/setup/platformSetup/index.website.ts
@@ -53,6 +53,7 @@ const webUpdater = (): PlatformSpecificUpdater => ({
 export default function () {
     AppRegistry.runApplication(Config.APP_NAME, {
         rootTag: document.getElementById('root'),
+        mode: 'legacy',
     });
 
     // When app loads, get current version (production only)