Use canNavigate instead of isReady

Expensify · Apr 4, 2022 · 2ef1277 · 2ef1277
1 parent 86050cc
commit 2ef1277
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/src/libs/Navigation/Navigation.js b/src/libs/Navigation/Navigation.js
@@ -31,14 +31,6 @@ function setDidTapNotification() {
     didTapNotificationBeforeReady = true;
 }
 
-/**
- * Returns true if the Navigation is ready to navigate
- * @returns {Boolean}
- */
-function isReady() {
-    return navigationRef.isReady();
-}
-
 /**
  * @param {String} methodName
  * @param {Object} params
@@ -225,11 +217,11 @@ DismissModal.defaultProps = {
 };
 
 export default {
+    canNavigate,
     navigate,
     dismissModal,
     isActiveRoute,
     getActiveRoute,
-    isReady,
     goBack,
     DismissModal,
     closeDrawer,

diff --git a/src/libs/actions/Report.js b/src/libs/actions/Report.js
@@ -797,7 +797,7 @@ function subscribeToReportCommentPushNotifications() {
 
     // Open correct report when push notification is clicked
     PushNotification.onSelected(PushNotification.TYPE.REPORT_COMMENT, ({reportID}) => {
-        if (Navigation.isReady()) {
+        if (Navigation.canNavigate('navigate')) {
             // If a chat is visible other than the one we are trying to navigate to, then we need to navigate back
             if (Navigation.getActiveRoute().slice(1, 2) === ROUTES.REPORT && !Navigation.isActiveRoute(`r/${reportID}`)) {
                 Navigation.goBack();