Merge branch 'main' of github.com:Expensify/App into jsenyitko-export-status-2

Author: JS00001

.github/ISSUE_TEMPLATE/NewPatchTemplate.md

@@ -0,0 +1,39 @@
+---
+name: Patch Tracking
+about: Template for tracking patches applied to third-party libraries
+---
+
+If you haven't already, check out our [contributing guidelines](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md) and [patch guidelines](https://github.com/Expensify/App/blob/main/contributingGuides/PATCHES.md) for information on how to manage patches.
+___
+
+## Patch Information
+
+**Library Name:** <!-- e.g., react-native-pdf -->
+**Library Version:** <!-- e.g., 6.7.3 -->
+**Patch Number:** <!-- e.g., 001 -->
+**Patch Description:** <!-- e.g., update-podspec-to-support-new-arch -->
+**Full Patch Filename:** <!-- e.g., react-native-pdf+6.7.3+001+update-podspec-to-support-new-arch.patch -->
+
+## Patch Details
+
+### Reason for Patch
+<!-- Explain why this patch is necessary. What issue does it solve? -->
+
+### Changes Made
+<!-- Briefly describe the changes made in the patch -->
+
+### Upstream Status
+**Upstream PR/Issue:** <!-- Link to the PR or issue in the upstream repository, if one exists -->
+
+## Related Information
+
+**PR Introducing Patch:** <!-- Link to the PR that introduced this patch -->
+
+## Additional Notes
+<!-- Any other relevant information about this patch -->
+
+## Checklist
+- [ ] Patch file is correctly named and placed in the appropriate directory
+- [ ] Patch is documented in the corresponding `details.md` file
+- [ ] This issue is linked in the `details.md` file
+- [ ] Upstream PR/issue has been created (if applicable)

.github/actions/javascript/checkTestCoverage/action.yml

@@ -0,0 +1,10 @@
+name: 'Check Test Coverage'
+description: 'A simple action that takes a GitHub URL and prints it'
+inputs:
+    GITHUB_URL:
+        description: 'GitHub repository URL to check'
+        required: true
+
+runs:
+    using: 'node20'
+    main: './index.js' 

.github/actions/javascript/checkTestCoverage/index.js

@@ -0,0 +1,17 @@
+const core = require('@actions/core');
+
+async function run() {
+    try {
+        // Get the GitHub URL input
+        const githubUrl = core.getInput('GITHUB_URL');
+        
+        // Print the URL
+        console.log(`GitHub URL: ${githubUrl}`);
+        core.info(`GitHub URL: ${githubUrl}`);
+        
+    } catch (error) {
+        core.setFailed(error.message);
+    }
+}
+
+run(); 

.github/workflows/check-test-coverage.yml

@@ -0,0 +1,22 @@
+name: Check Test Coverage
+
+on:
+  workflow_dispatch:
+    inputs:
+      github_url:
+        description: 'GitHub repository URL to check'
+        required: true
+        type: string
+
+jobs:
+  check-coverage:
+    runs-on: ubuntu-latest
+    name: Check Test Coverage
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        
+      - name: Check Test Coverage
+        uses: ./.github/actions/javascript/checkTestCoverage
+        with:
+          GITHUB_URL: ${{ inputs.github_url }}

.storybook/webpackMockPaths.ts

@@ -6,6 +6,5 @@ export default {
     'react-native$': 'react-native-web',
     '@react-native-community/netinfo': path.resolve(__dirname, '../__mocks__/@react-native-community/netinfo.ts'),
     '@react-navigation/native': path.resolve(__dirname, '../__mocks__/@react-navigation/native'),
-    '@libs/TransactionPreviewUtils': path.resolve(__dirname, '../src/libs/__mocks__/TransactionPreviewUtils.ts'),
 };
 /* eslint-enable @typescript-eslint/naming-convention */

Mobile-Expensify

@@ -8,6 +8,7 @@ const realReactNavigation = isJestEnv ? jest.requireActual<typeof ReactNavigatio
 
 const useIsFocused = isJestEnv ? realReactNavigation.useIsFocused : () => true;
 const useTheme = isJestEnv ? realReactNavigation.useTheme : () => ({});
+const useLocale = isJestEnv ? realReactNavigation.useTheme : () => ({});
 
 const {triggerTransitionEnd, addListener} = isJestEnv
     ? createAddListenerMock()
@@ -44,14 +45,14 @@ const useLinkTo = isJestEnv ? realReactNavigation.useLinkTo : () => null;
 const useScrollToTop = isJestEnv ? realReactNavigation.useScrollToTop : () => null;
 const useRoute = isJestEnv ? realReactNavigation.useRoute : () => ({params: {}});
 const useFocusEffect = isJestEnv ? realReactNavigation.useFocusEffect : (callback: () => void) => callback();
-// eslint-disable-next-line @typescript-eslint/naming-convention
-const UNSTABLE_usePreventRemove = isJestEnv ? jest.fn() : () => {};
+const usePreventRemove = isJestEnv ? jest.fn() : () => {};
 
 export {
     // Overridden modules
     useIsFocused,
     useTheme,
     useNavigation,
+    useLocale,
     triggerTransitionEnd,
 
     // Theme modules are left alone
@@ -68,7 +69,7 @@ export {
     useScrollToTop,
     useRoute,
     useFocusEffect,
-    UNSTABLE_usePreventRemove,
+    usePreventRemove,
 };
 
 export type {NativeNavigationMock};

__mocks__/@react-navigation/native/index.ts

@@ -114,8 +114,8 @@ android {
         minSdkVersion rootProject.ext.minSdkVersion
         targetSdkVersion rootProject.ext.targetSdkVersion
         multiDexEnabled rootProject.ext.multiDexEnabled
-        versionCode 1009017104
-        versionName "9.1.71-4"
+        versionCode 1009017300
+        versionName "9.1.73-0"
         // Supported language variants must be declared here to avoid from being removed during the compilation.
         // This also helps us to not include unnecessary language variants in the APK.
         resConfigs "en", "es"

android/app/build.gradle

@@ -33,6 +33,10 @@ const includeModules = [
     'react-native-web',
     'react-native-webview',
     '@react-native-picker',
+    '@react-navigation/material-top-tabs',
+    '@react-navigation/native',
+    '@react-navigation/native-stack',
+    '@react-navigation/stack',
     'react-native-modal',
     'react-native-gesture-handler',
     'react-native-google-places-autocomplete',
@@ -160,6 +164,12 @@ const getCommonConfiguration = ({file = '.env', platform = 'web'}: Environment):
     ],
     module: {
         rules: [
+            {
+                test: /\.m?js$/,
+                resolve: {
+                    fullySpecified: false,
+                },
+            },
             // Transpiles and lints all the JS
             {
                 test: /\.(js|ts)x?$/,

config/webpack/webpack.common.ts

@@ -6,6 +6,34 @@ To ensure patches remain maintainable and traceable over time, follow the guidel
 
 ---
 
+## 🛠️ Creating a Patch
+
+To create a patch for a third-party library that you've modified:
+
+1. Make your changes to the library code in the `node_modules/<edited-library>` directory
+2. Run the following command to generate the patch file:
+
+```
+npx patch-package <edited-library> --append "<short-patch-description>" --patch-dir ./patches/<edited-library>
+```
+
+For example:
+```
+npx patch-package react-native-pdf --append "fix-pdf-rendering-on-ios" --patch-dir ./patches/react-native-pdf
+```
+
+This will create a patch file in the `patches/` directory. After creating the patch:
+
+- Create or update the `details.md` file in the library's patch directory
+- Document the patch according to the [`details.md` format](#-detailsmd-format)
+
+> 📝 **Note**
+>
+> The `--patch-dir` option should only be specified when the target library has a dedicated subdirectory within the `./patches` directory.
+> If no such subdirectory exists, omit this option from the command.
+
+---
+
 ## 🗂️ Folder Structure
 
 Each library with patch (or patches) should have its own directory inside `patches/`. Each directory should contain:
@@ -54,23 +82,34 @@ Each patch must be listed and explained in the `details.md` file within the same
 
 ### [<patch-name>.patch](<patch-name>.patch)
 
-- Reason: <Why this patch is needed>
-- Upstream PR/issue: <link or 🛑 if not raised. If no upstream issue or PR exists, explain why>
-- E/App issue: <link or 🛑 if none>
-- PR Introducing Patch: <link to internal PR that added the patch>
+- Reason:
+  
+    ```
+    Please explain why the patch is necessary
+    ```
+  
+- Upstream PR/issue: <Please create an upstream PR fixing the patch. Link it here and if no upstream issue or PR exists, explain why>
+- E/App issue: <Please create an E/App issue ([template](./../.github/ISSUE_TEMPLATE/NewPatchTemplate.md)) for each introduced patch. Link it here and if patch won't be removed in the future (no upstream PR exists) explain why>
+- PR introducing patch: <Link to E/App (or Mobile-Expensify) PR that added the patch>
 ```
 
 ### Example
 
 ```md
 # `react-native-pdf` patches
 
-### [react-native-pdf+6.7.3+002+fix-incorrect-decoding.patch](react-native-pdf+6.7.3+002+fix-incorrect-decoding.patch)
+### [react-native-pdf+6.7.3+001+update-podspec-to-support-new-arch.patch](react-native-pdf+6.7.3+001+update-podspec-to-support-new-arch.patch)
+
+- Reason:
+
+    ```
+    This patch updates the react-native-pdf.podspec to ensure compatibility with React Native's New Architecture on iOS by replacing manual dependency declarations
+    with Meta's recommended `install_modules_dependencies` function
+    ```
 
-- Reason: If the file name contains accented characters, the PDF load fails.
-- Upstream PR/issue: 🛑
-- E/App issue: 🛑
-- PR Introducing Patch: [#50043](https://github.com/Expensify/App/pull/50043)
+- Upstream PR/issue: https://github.com/wonday/react-native-pdf/pull/803
+- E/App issue: 🛑 TODO
+- PR Introducing Patch: https://github.com/Expensify/App/pull/13767
 ```
 
 ## ✅ Patch Submission Checklist