From 3d5bc852356f98109d91b64bcb4b42c0548859de Mon Sep 17 00:00:00 2001
From: Peter Velkov <kidroca@gmail.com>
Date: Wed, 14 Jun 2023 15:53:47 +0300
Subject: [PATCH] Refactor: extract "Report_Attachments" string to SCREENS

---
 src/ROUTES.js                                   | 2 +-
 src/SCREENS.js                                  | 1 +
 src/libs/Navigation/AppNavigator/AuthScreens.js | 2 +-
 src/libs/Navigation/Navigation.js               | 3 ++-
 src/libs/Navigation/linkingConfig.js            | 2 +-
 5 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/ROUTES.js b/src/ROUTES.js
index ad8608593614..4c3f24b08bd4 100644
--- a/src/ROUTES.js
+++ b/src/ROUTES.js
@@ -68,7 +68,7 @@ export default {
     getReportRoute: (reportID) => `r/${reportID}`,
     REPORT_WITH_ID_DETAILS_SHARE_CODE: 'r/:reportID/details/shareCode',
     getReportShareCodeRoute: (reportID) => `r/${reportID}/details/shareCode`,
-    REPORT_ATTACHMENT: 'r/:reportID/attachment',
+    REPORT_ATTACHMENTS: 'r/:reportID/attachment',
     getReportAttachmentRoute: (reportID, source) => `r/${reportID}/attachment?source=${encodeURI(source)}`,
     SELECT_YEAR: 'select-year',
     getYearSelectionRoute: (minYear, maxYear, currYear, backTo) => `select-year?min=${minYear}&max=${maxYear}&year=${currYear}&backTo=${backTo}`,
diff --git a/src/SCREENS.js b/src/SCREENS.js
index 24ea27fe9689..aec13a490376 100644
--- a/src/SCREENS.js
+++ b/src/SCREENS.js
@@ -6,6 +6,7 @@ export default {
     HOME: 'Home',
     LOADING: 'Loading',
     REPORT: 'Report',
+    REPORT_ATTACHMENTS: 'ReportAttachments',
     NOT_FOUND: 'not-found',
     TRANSITION_FROM_OLD_DOT: 'TransitionFromOldDot',
 };
diff --git a/src/libs/Navigation/AppNavigator/AuthScreens.js b/src/libs/Navigation/AppNavigator/AuthScreens.js
index 735f531bcaef..edac1d1638bf 100644
--- a/src/libs/Navigation/AppNavigator/AuthScreens.js
+++ b/src/libs/Navigation/AppNavigator/AuthScreens.js
@@ -251,7 +251,7 @@ class AuthScreens extends React.Component {
                     }}
                 />
                 <RootStack.Screen
-                    name="Report_Attachments"
+                    name={SCREENS.REPORT_ATTACHMENTS}
                     options={{
                         headerShown: false,
                         presentation: 'transparentModal',
diff --git a/src/libs/Navigation/Navigation.js b/src/libs/Navigation/Navigation.js
index cf9b7b979daf..6640095009fe 100644
--- a/src/libs/Navigation/Navigation.js
+++ b/src/libs/Navigation/Navigation.js
@@ -11,6 +11,7 @@ import navigationRef from './navigationRef';
 import NAVIGATORS from '../../NAVIGATORS';
 import originalGetTopmostReportId from './getTopmostReportId';
 import getStateFromPath from './getStateFromPath';
+import SCREENS from '../../SCREENS';
 
 let resolveNavigationIsReadyPromise;
 const navigationIsReadyPromise = new Promise((resolve) => {
@@ -135,7 +136,7 @@ function dismissModal(targetReportID) {
     switch (lastRoute.name) {
         case NAVIGATORS.RIGHT_MODAL_NAVIGATOR:
         case NAVIGATORS.FULL_SCREEN_NAVIGATOR:
-        case 'Report_Attachments':
+        case SCREENS.REPORT_ATTACHMENTS:
             // if we are not in the target report, we need to navigate to it after dismissing the modal
             if (targetReportID && targetReportID !== getTopmostReportId(rootState)) {
                 const state = getStateFromPath(ROUTES.getReportRoute(targetReportID));
diff --git a/src/libs/Navigation/linkingConfig.js b/src/libs/Navigation/linkingConfig.js
index a4a142658236..33a35fc25e69 100644
--- a/src/libs/Navigation/linkingConfig.js
+++ b/src/libs/Navigation/linkingConfig.js
@@ -14,7 +14,7 @@ export default {
             UnlinkLogin: ROUTES.UNLINK_LOGIN,
             [SCREENS.TRANSITION_FROM_OLD_DOT]: ROUTES.TRANSITION_FROM_OLD_DOT,
             Concierge: ROUTES.CONCIERGE,
-            Report_Attachments: ROUTES.REPORT_ATTACHMENT,
+            [SCREENS.REPORT_ATTACHMENTS]: ROUTES.REPORT_ATTACHMENTS,
 
             // Sidebar
             [SCREENS.HOME]: {