From 58647a348c51022b7dd537aab457c5d75e63fc71 Mon Sep 17 00:00:00 2001
From: SpyTec <eric@spytec.se>
Date: Sun, 8 Oct 2023 00:37:15 +0200
Subject: [PATCH] feat: moved hash to results link

---
 .../app/Results/ResultsPageTestingFarm.tsx    | 50 +++++++++++--------
 1 file changed, 28 insertions(+), 22 deletions(-)

diff --git a/frontend/src/app/Results/ResultsPageTestingFarm.tsx b/frontend/src/app/Results/ResultsPageTestingFarm.tsx
index f6b22fd..b4322a3 100644
--- a/frontend/src/app/Results/ResultsPageTestingFarm.tsx
+++ b/frontend/src/app/Results/ResultsPageTestingFarm.tsx
@@ -97,7 +97,6 @@ function useCoprBuilds(copr_build_ids: number[]): React.JSX.Element[] {
     const coprDetails = results.map((result) => {
         if (!result.data || typeof result.data !== "object")
             return <>Loading</>;
-        console.log(result.data);
         if ("error" in result.data) {
             return <>Error {result.data["error"]}</>;
         } else if ("build_id" in result.data) {
@@ -195,6 +194,12 @@ const ResultsPageTestingFarm = () => {
         <>{data?.status}</>
     );
 
+    const onCopyHash = () => {
+        if (data) {
+            navigator.clipboard.writeText(data.commit_sha.substring(0, 7));
+        }
+    };
+
     return (
         <>
             <PageSection variant={PageSectionVariants.light}>
@@ -203,7 +208,28 @@ const ResultsPageTestingFarm = () => {
 
                     <Text component="p">
                         <strong>
-                            {data ? <TriggerLink builds={data} /> : <></>}
+                            {data ? (
+                                <>
+                                    <TriggerLink builds={data} />
+                                    <ClipboardCopy
+                                        variant="inline-compact"
+                                        onCopy={onCopyHash}
+                                    >
+                                        <a
+                                            href={getCommitLink(
+                                                data.git_repo,
+                                                data.commit_sha,
+                                            )}
+                                            rel="noreferrer"
+                                            target="_blank"
+                                        >
+                                            {data.commit_sha.substring(0, 7)}
+                                        </a>
+                                    </ClipboardCopy>
+                                </>
+                            ) : (
+                                <></>
+                            )}
                         </strong>
                         <br />
                     </Text>
@@ -260,26 +286,6 @@ const ResultsPageTestingFarm = () => {
                                             </ClipboardCopy>
                                         </DescriptionListDescription>
                                     </DescriptionListGroup>
-                                    <DescriptionListGroup>
-                                        <DescriptionListTerm>
-                                            Commit SHA
-                                        </DescriptionListTerm>
-                                        <DescriptionListDescription>
-                                            <a
-                                                href={getCommitLink(
-                                                    data.git_repo,
-                                                    data.commit_sha,
-                                                )}
-                                                rel="noreferrer"
-                                                target="_blank"
-                                            >
-                                                {data.commit_sha.substring(
-                                                    0,
-                                                    7,
-                                                )}
-                                            </a>
-                                        </DescriptionListDescription>
-                                    </DescriptionListGroup>
                                 </DescriptionList>
                             </CardBody>
                             <CardBody>