From 90a160e83fdbcdf1ae690191b0519a120e94f09e Mon Sep 17 00:00:00 2001
From: Ali <83188384+testA113@users.noreply.github.com>
Date: Thu, 7 Dec 2023 17:05:17 +1300
Subject: [PATCH] Revert "fix(gitops): correct commit hash link [EE-6346]
 (#10722)" (#10788)

This reverts commit 83cd5d9b2fb4a0c9ef163ae6e229587b27c07999.
---
 .../ItemView/EnvironmentsDatatable/columns.tsx           | 9 ++-------
 .../edge-stacks/ListView/EdgeStacksDatatable/columns.tsx | 5 +----
 app/react/edge/edge-stacks/utils.ts                      | 5 -----
 3 files changed, 3 insertions(+), 16 deletions(-)

diff --git a/app/react/edge/edge-stacks/ItemView/EnvironmentsDatatable/columns.tsx b/app/react/edge/edge-stacks/ItemView/EnvironmentsDatatable/columns.tsx
index f7702b7f1..094f873f6 100644
--- a/app/react/edge/edge-stacks/ItemView/EnvironmentsDatatable/columns.tsx
+++ b/app/react/edge/edge-stacks/ItemView/EnvironmentsDatatable/columns.tsx
@@ -15,7 +15,6 @@ import { Icon } from '@@/Icon';
 import { Link } from '@@/Link';
 
 import { DeploymentStatus, EdgeStackStatus, StatusType } from '../../types';
-import { removeTrailingGitExtension } from '../../utils';
 
 import { EnvironmentActions } from './EnvironmentActions';
 import { ActionStatus } from './ActionStatus';
@@ -188,9 +187,7 @@ function TargetVersionCell({
       {row.original.TargetCommitHash ? (
         <div>
           <a
-            href={`${removeTrailingGitExtension(
-              row.original.GitConfigURL
-            )}/commit/${row.original.TargetCommitHash}`}
+            href={`${row.original.GitConfigURL}/commit/${row.original.TargetCommitHash}`}
             target="_blank"
             rel="noreferrer"
           >
@@ -239,9 +236,7 @@ function DeployedVersionCell({
         <div>
           {statusIcon}
           <a
-            href={`${removeTrailingGitExtension(
-              row.original.GitConfigURL
-            )}/commit/${row.original.TargetCommitHash}`}
+            href={`${row.original.GitConfigURL}/commit/${row.original.TargetCommitHash}`}
             target="_blank"
             rel="noreferrer"
           >
diff --git a/app/react/edge/edge-stacks/ListView/EdgeStacksDatatable/columns.tsx b/app/react/edge/edge-stacks/ListView/EdgeStacksDatatable/columns.tsx
index 5452d926f..354b9669a 100644
--- a/app/react/edge/edge-stacks/ListView/EdgeStacksDatatable/columns.tsx
+++ b/app/react/edge/edge-stacks/ListView/EdgeStacksDatatable/columns.tsx
@@ -8,7 +8,6 @@ import { buildNameColumn } from '@@/datatables/NameCell';
 import { Link } from '@@/Link';
 
 import { StatusType } from '../../types';
-import { removeTrailingGitExtension } from '../../utils';
 
 import { EdgeStackStatus } from './EdgeStacksStatus';
 import { DecoratedEdgeStack } from './types';
@@ -152,9 +151,7 @@ export const columns = _.compact([
               <div className="text-center">
                 <a
                   target="_blank"
-                  href={`${removeTrailingGitExtension(
-                    item.GitConfig.URL
-                  )}/commit/${item.GitConfig.ConfigHash}`}
+                  href={`${item.GitConfig.URL}/commit/${item.GitConfig.ConfigHash}`}
                   rel="noreferrer"
                 >
                   {item.GitConfig.ConfigHash.slice(0, 7)}
diff --git a/app/react/edge/edge-stacks/utils.ts b/app/react/edge/edge-stacks/utils.ts
index 5c4bedee2..82cbfec7b 100644
--- a/app/react/edge/edge-stacks/utils.ts
+++ b/app/react/edge/edge-stacks/utils.ts
@@ -19,8 +19,3 @@ export function getValidEditorTypes(
     ? _.intersection(...endpointTypes.map((type) => right[type]))
     : [EditorType.Compose, EditorType.Kubernetes];
 }
-
-export function removeTrailingGitExtension(url: string) {
-  // the url could have the .git extension. Remove it if it does.
-  return url.replace(/\.git$/, '');
-}