From 7c3d4da8508f101b2c912d8750a99fa79be90802 Mon Sep 17 00:00:00 2001
From: Andrew Farries <andrew@gitpod.io>
Date: Wed, 27 Jul 2022 06:48:34 +0000
Subject: [PATCH] Hide PVC feature section when feat flag not set

Rather than having the section always visible but with a disable
checkbox, hide the entire section if the feature flag is not set.
---
 .../src/projects/ProjectSettings.tsx          | 35 ++++++++++---------
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/components/dashboard/src/projects/ProjectSettings.tsx b/components/dashboard/src/projects/ProjectSettings.tsx
index 670f9dc0ee8e08..955b634cedda2b 100644
--- a/components/dashboard/src/projects/ProjectSettings.tsx
+++ b/components/dashboard/src/projects/ProjectSettings.tsx
@@ -94,22 +94,25 @@ export default function () {
                 checked={!project.settings?.keepOutdatedPrebuildsRunning}
                 onChange={({ target }) => updateProjectSettings({ keepOutdatedPrebuildsRunning: !target.checked })}
             />
-            <br></br>
-            <h3 className="mt-12">Workspace Persistence</h3>
-            <CheckBox
-                title={
-                    <span>
-                        Enable Persistent Volume Claim{" "}
-                        <PillLabel type="warn" className="font-semibold mt-2 ml-2 py-0.5 px-2 self-center">
-                            Experimental
-                        </PillLabel>
-                    </span>
-                }
-                desc={<span>Experimental feature that is still under development.</span>}
-                checked={project.settings?.usePersistentVolumeClaim ?? false}
-                disabled={!showPersistentVolumeClaimUI}
-                onChange={({ target }) => updateProjectSettings({ usePersistentVolumeClaim: target.checked })}
-            />
+            {showPersistentVolumeClaimUI && (
+                <>
+                    <br></br>
+                    <h3 className="mt-12">Workspace Persistence</h3>
+                    <CheckBox
+                        title={
+                            <span>
+                                Enable Persistent Volume Claim{" "}
+                                <PillLabel type="warn" className="font-semibold mt-2 ml-2 py-0.5 px-2 self-center">
+                                    Experimental
+                                </PillLabel>
+                            </span>
+                        }
+                        desc={<span>Experimental feature that is still under development.</span>}
+                        checked={project.settings?.usePersistentVolumeClaim ?? false}
+                        onChange={({ target }) => updateProjectSettings({ usePersistentVolumeClaim: target.checked })}
+                    />
+                </>
+            )}
         </ProjectSettingsPage>
     );
 }