Add navigateToScreen as dependency

navigateToScreen should be added as the hook dependency, or hasEdits might not be the latest value when calling the function. In fact, due to the property of how hasEdits is updated, the navigateToScreen would always get the latest value of hasEdits, but it's best practice to add state and props that are used in the hook as its dependency, or it might get harder to develop new feature or maintain the code someday in the future.

settings/src/components/revalidate-modal.js

@@ -10,10 +10,13 @@ import { refreshRecord } from '../utilities/common';
 export default function RevalidateModal() {
 	const { navigateToScreen } = useContext( GlobalContext );
 
-	const goBack = useCallback( ( event ) => {
-		event.preventDefault();
-		navigateToScreen( 'account-status' );
-	}, [] );
+	const goBack = useCallback(
+		( event ) => {
+			event.preventDefault();
+			navigateToScreen( 'account-status' );
+		},
+		[ navigateToScreen ]
+	);
 
 	return (
 		<Modal

settings/src/components/screen-link.js

@@ -23,10 +23,13 @@ export default function ScreenLink( { screen, anchorText, buttonStyle = false, a
 		classes.push( 'is-secondary' );
 	}
 
-	const onClick = useCallback( ( event ) => {
-		event.preventDefault();
-		navigateToScreen( screen );
-	}, [] );
+	const onClick = useCallback(
+		( event ) => {
+			event.preventDefault();
+			navigateToScreen( screen );
+		},
+		[ navigateToScreen ]
+	);
 
 	return (
 		<a