fix: trim ref/heads/ only from the prefix

Do the same for the trailing .git for the ref/heads/ prefix to
be safe and clean things up.

Author: 06kellyjac

src/db/helper.ts

@@ -13,3 +13,11 @@ export const trimTrailingDotGit = (str: string): string => {
   return str;
 };
 
+export const trimPrefixRefsHeads = (str: string): string => {
+  const target = 'refs/heads/';
+  if (str.startsWith(target)) {
+    // extract string from the end of the target to the end of str
+    return str.slice(target.length);
+  }
+  return str;
+};

src/ui/views/OpenPushRequests/components/PushesTable.tsx

@@ -16,7 +16,7 @@ import { KeyboardArrowRight } from '@material-ui/icons';
 import Search from '../../../components/Search/Search';
 import Pagination from '../../../components/Pagination/Pagination';
 import { PushData } from '../../../../types/models';
-import { trimTrailingDotGit } from '../../../../db/helper';
+import { trimPrefixRefsHeads, trimTrailingDotGit } from '../../../../db/helper';
 
 interface PushesTableProps {
   [key: string]: any;
@@ -102,7 +102,7 @@ const PushesTable: React.FC<PushesTableProps> = (props) => {
           <TableBody>
             {[...currentItems].reverse().map((row) => {
               const repoFullName = trimTrailingDotGit(row.repo);
-              const repoBranch = row.branch.replace('refs/heads/', '');
+              const repoBranch = trimPrefixRefsHeads(row.branch);
               const commitTimestamp =
                 row.commitData[0]?.commitTs || row.commitData[0]?.commitTimestamp;
 

src/ui/views/PushDetails/PushDetails.tsx

@@ -23,7 +23,7 @@ import { CheckCircle, Visibility, Cancel, Block } from '@material-ui/icons';
 import Snackbar from '@material-ui/core/Snackbar';
 import Tooltip from '@material-ui/core/Tooltip';
 import { PushData } from '../../../types/models';
-import { trimTrailingDotGit } from '../../../db/helper';
+import { trimPrefixRefsHeads, trimTrailingDotGit } from '../../../db/helper';
 
 const Dashboard: React.FC = () => {
   const { id } = useParams<{ id: string }>();
@@ -107,7 +107,7 @@ const Dashboard: React.FC = () => {
   }
 
   const repoFullName = trimTrailingDotGit(data.repo);
-  const repoBranch = data.branch.replace('refs/heads/', '');
+  const repoBranch = trimPrefixRefsHeads(data.branch);
 
   const generateIcon = (title: string) => {
     switch (title) {