fix(graph): encode id in project node tooltip (#14530)

nrwl · Jan 23, 2023 · d43d6b6 · d43d6b6 · vercel · Jan 23, 2023
1 parent e7f065e
commit d43d6b6
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/graph/client/src/app/ui-tooltips/project-node-actions.tsx b/graph/client/src/app/ui-tooltips/project-node-actions.tsx
@@ -11,6 +11,7 @@ export function ProjectNodeActions({ id }: ProjectNodeToolTipProps) {
     projectGraphService.getSnapshot().context.tracing;
   const routeConstructor = useRouteConstructor();
   const navigate = useNavigate();
+  const encodedId = encodeURIComponent(id);
 
   function onExclude() {
     projectGraphService.send({
@@ -21,25 +22,21 @@ export function ProjectNodeActions({ id }: ProjectNodeToolTipProps) {
   }
 
   function onStartTrace() {
-    navigate(
-      routeConstructor(`/projects/trace/${encodeURIComponent(id)}`, true)
-    );
+    navigate(routeConstructor(`/projects/trace/${encodedId}`, true));
   }
 
   function onEndTrace() {
     navigate(
       routeConstructor(
-        `/projects/trace/${encodeURIComponent(start)}/${encodeURIComponent(
-          id
-        )}`,
+        `/projects/trace/${encodeURIComponent(start)}/${encodedId}`,
         true
       )
     );
   }
 
   return (
     <div className="grid grid-cols-3 gap-4">
-      <TooltipLinkButton to={routeConstructor(`/projects/${id}`, true)}>
+      <TooltipLinkButton to={routeConstructor(`/projects/${encodedId}`, true)}>
         Focus
       </TooltipLinkButton>
       <TooltipButton onClick={onExclude}>Exclude</TooltipButton>