From aae3f4301bffa152b2dd6330e12f39fed0571ebd Mon Sep 17 00:00:00 2001 From: Abdul Rahman Date: Wed, 8 Apr 2026 20:35:36 +0530 Subject: [PATCH] Remove unused `handleBeforeNavigation` call in `NavigationDrawerItem` component - Eliminated the `handleBeforeNavigation` function call from the `handleExternalLinkClick` method to streamline the navigation logic. - This change enhances code clarity by removing unnecessary function calls, contributing to improved maintainability of the component. These updates simplify the navigation handling within the `NavigationDrawerItem`, aligning with recent refactoring efforts. --- .../navigation-drawer/components/NavigationDrawerItem.tsx | 1 - 1 file changed, 1 deletion(-) diff --git a/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawerItem.tsx b/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawerItem.tsx index 2a68da1c9e5..886e843ac85 100644 --- a/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawerItem.tsx +++ b/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawerItem.tsx @@ -300,7 +300,6 @@ export const NavigationDrawerItem = ({ const isInternalLink = isDefined(to) && !isExternalLink; const handleExternalLinkClick = () => { - handleBeforeNavigation(); if (isDefined(to)) { window.open(to, '_blank', 'noopener,noreferrer'); }