From 9604aed7e54c40f91ffd91bd9919dfa8e7c6a8d7 Mon Sep 17 00:00:00 2001
From: Laszlo Fogas <laszlo@gimlet.io>
Date: Wed, 16 Oct 2024 09:40:10 +0200
Subject: [PATCH] Making sourceRef linkable

---
 web/src/HelmRevisionWidget.jsx | 23 ++++++++++-------------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/web/src/HelmRevisionWidget.jsx b/web/src/HelmRevisionWidget.jsx
index 1f00085..cd46d63 100644
--- a/web/src/HelmRevisionWidget.jsx
+++ b/web/src/HelmRevisionWidget.jsx
@@ -25,7 +25,6 @@ export function HelmRevisionWidget(props) {
   const reconciling = reconcilingCondition && reconcilingConditions[0].status === "True"
 
   const sourceRef = helmRelease.spec.chart ? helmRelease.spec.chart.spec.sourceRef : helmRelease.spec.chartRef
-
   const namespace = sourceRef.namespace ? sourceRef.namespace : helmRelease.metadata.namespace
   const navigationHandler = () => handleNavigationSelect("Sources", namespace, sourceRef.name, sourceRef.kind)
 
@@ -33,30 +32,28 @@ export function HelmRevisionWidget(props) {
     <>
       {!ready && reconciling && !stalled &&
         <span>
-          {helmRelease.spec.chart &&
-          <>
           <span>Reconciling new version: </span>
+          <NavigationButton handleNavigation={navigationHandler}>
+          {helmRelease.spec.chart &&
           <span>{helmRelease.spec.chart.spec.version}@{helmRelease.spec.chart.spec.chart}</span>
-          </>
           }
-          {!helmRelease.spec.chart &&
-            <span>Reconciling new version..</span> // chartRef doesn't have version info
+          {helmRelease.spec.chartRef &&
+          <span>{`${helmRelease.spec.chartRef.namespace}/${helmRelease.spec.chartRef.name}`}</span>
           }
+          </NavigationButton>
         </span>
       }
       {!ready && stalled &&
         <span className='bg-orange-400'>
+          <span>Reconciliation stalled: </span>
+          <NavigationButton handleNavigation={navigationHandler}>
           {helmRelease.spec.chart &&
-          <>
-          <span>Last Attempted: </span>
           <span>{helmRelease.spec.chart.spec.version}@{helmRelease.spec.chart.spec.chart}</span>
-          </>
           }
-          {!helmRelease.spec.chart &&
-          <>
-            <span>Reconciliation stalled..</span>  // chartRef doesn't have version info
-          </>
+          {helmRelease.spec.chartRef &&
+          <span>{`${helmRelease.spec.chartRef.namespace}/${helmRelease.spec.chartRef.name}`}</span>
           }
+          </NavigationButton>
         </span>
       }
       <span className={`block ${ready || reconciling ? '' : 'font-normal text-neutral-600'} field`}>