Feature implementation from commits 7ce106a..f120b09

[codeOwlAI]

PR Summary

Add useLocationBlocker Hook to Prevent Duplicate History Entries

Overview

This PR introduces a new custom hook useLocationBlocker that prevents React Router from pushing the same page to history multiple times, improving navigation experience by avoiding duplicate history entries.

Change Types

Type	Description
Enhancement	Added a custom hook to improve navigation behavior
Refactor	Updated routing components to use the new hook

---

Affected Modules

Module / File	Change Description
shared/functions/useLocationBlocker.js	Added new custom hook to prevent duplicate history entries
logged_in/components/Routing.js	Implemented useLocationBlocker hook
logged_out/components/Routing.js	Implemented useLocationBlocker hook

Notes for Reviewers

• This change affects navigation behavior by preventing duplicate history entries
• Improves user experience by avoiding multiple back-button clicks to exit the application

[codeowlai]

🐛 Correctness Issue

Memory Leak: Missing Cleanup Function.

The useEffect hook doesn't return the cleanup function from history.block(), which will cause memory leaks when components unmount.

Current Code (Diff):

-  useEffect(
-    () =>
-      history.block(
-        (location, action) =>
-          action !== "PUSH" ||
-          getLocationId(location) !== getLocationId(history.location)
-      ),
-    [] // eslint-disable-line react-hooks/exhaustive-deps
-  );
+  useEffect(
+    () => {
+      const unblock = history.block(
+        (location, action) =>
+          action !== "PUSH" ||
+          getLocationId(location) !== getLocationId(history.location)
+      );
+      return unblock;
+    },
+    [] // eslint-disable-line react-hooks/exhaustive-deps
+  );

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	useEffect(
	() =>
	history.block(
	(location, action) =>
	action !== "PUSH" ||
	getLocationId(location) !== getLocationId(history.location)
	),
	[] // eslint-disable-line react-hooks/exhaustive-deps
	);
	useEffect(
	() => {
	const unblock = history.block(
	(location, action) =>
	action !== "PUSH" ||
	getLocationId(location) !== getLocationId(history.location)
	);
	return unblock;
	},
	[] // eslint-disable-line react-hooks/exhaustive-deps
	);