Fix PR comments

Author: brian-kim31

app/next-client-app/app/(protected)/datasets/columns.tsx

@@ -69,7 +69,13 @@ export const columns: ColumnDef<DataSet>[] = [
       />
     ),
     enableHiding: true,
-    enableSorting: true
+    enableSorting: true,
+    // Show Shared Visibility or Restricted Visibility
+    cell: ({ row }) => (
+      <span>
+        {row.original.visibility === "PUBLIC" ? "Shared" : "Restricted"}
+      </span>
+    )
   },
 
   {

app/next-client-app/components/datasets/CreateDatasetForm.tsx

@@ -101,7 +101,7 @@ export function CreateDatasetForm({
           viewers: [],
           editors: [],
           admins: [],
-          visibility: "shared",
+          visibility: "PUBLIC", // Always use "PUBLIC" or "RESTRICTED" for backend compatibility
           name: "",
           projects: 0,
         }}
@@ -178,15 +178,16 @@ export function CreateDatasetForm({
                     handleChange({
                       target: {
                         name: "visibility",
-                        value: checked ? "shared" : "restricted",
+                        value: checked ? "PUBLIC" : "RESTRICTED",
                       },
                     });
                     setPublicVisibility(checked);
                   }}
                   defaultChecked
                 />
                 <Label className="text-lg">
-                  {values.visibility === "shared" ? "shared" : "restricted"}
+                  {/* Show user-friendly label */}
+                  {values.visibility === "PUBLIC" ? "Shared" : "Restricted"}
                 </Label>
               </div>
               {!publicVisibility && (

app/next-client-app/components/datasets/DatasetForm.tsx

@@ -40,7 +40,7 @@ export function DatasetForm({
   const canUpdate = permissions.includes("CanAdmin");
   // State control for viewers fields
   const [publicVisibility, setPublicVisibility] = useState<boolean>(
-    dataset.visibility === "shared" ? true : false
+    dataset.visibility === "PUBLIC" ? true : false
   );
 
   // Making options suitable for React Select
@@ -83,7 +83,7 @@ export function DatasetForm({
     <Formik
       initialValues={{
         name: dataset.name,
-        visibility: dataset.visibility,
+        visibility: dataset.visibility, // Should be "PUBLIC" or "RESTRICTED"
         viewers: initialViewersFilter.map((viewer) => viewer.value),
         editors: initialEditorsFilter.map((editor) => editor.value),
         dataPartner: initialPartnerFilter[0].value,
@@ -134,16 +134,17 @@ export function DatasetForm({
                   handleChange({
                     target: {
                       name: "visibility",
-                      value: checked ? "shared" : "restricted",
+                      value: checked ? "PUBLIC" : "RESTRICTED",
                     },
                   });
                   setPublicVisibility(checked);
                 }}
-                defaultChecked={dataset.visibility === "shared" ? true : false}
+                defaultChecked={dataset.visibility === "PUBLIC" ? true : false}
                 disabled={!canUpdate}
               />
               <Label className="text-lg">
-                {values.visibility === "shared" ? "shared" : "restricted"}
+                {/* Show user-friendly label */}
+                {values.visibility === "PUBLIC" ? "Shared" : "Restricted"}
               </Label>
             </div>
             {!publicVisibility && (

app/next-client-app/components/scanreports/CreateScanReportForm.tsx

@@ -94,7 +94,7 @@ export function CreateScanReportForm({
           dataset: 0,
           viewers: [],
           editors: [],
-          visibility: "shared",
+          visibility: "PUBLIC",
           name: "",
           scan_report_file: null,
           Data_dict: null
@@ -189,15 +189,16 @@ export function CreateScanReportForm({
                     handleChange({
                       target: {
                         name: "visibility",
-                        value: checked ? "shared" : "restricted"
+                        value: checked ? "PUBLIC" : "RESTRICTED"
                       }
                     });
                     setPublicVisibility(checked);
                   }}
-                  defaultChecked
+                  checked={values.visibility === "PUBLIC"}
                 />
                 <Label className="text-lg">
-                  {values.visibility === "shared" ? "shared" : "restricted"}
+                  {/* Show user-friendly label */}
+                  {values.visibility === "PUBLIC" ? "Shared" : "Restricted"}
                 </Label>
               </div>
               {!publicVisibility && (

app/next-client-app/components/scanreports/ScanReportDetailsForm.tsx

@@ -39,7 +39,7 @@ export function ScanReportDetailsForm({
   const canUpdate = permissions.includes("CanAdmin") || isAuthor;
   // State control for viewers fields
   const [publicVisibility, setPublicVisibility] = useState<boolean>(
-    scanreport.visibility === "shared" ? true : false
+    scanreport.visibility === "PUBLIC" ? true : false
   );
 
   // Making options suitable for React Select
@@ -140,18 +140,17 @@ export function ScanReportDetailsForm({
                   handleChange({
                     target: {
                       name: "visibility",
-                      value: checked ? "shared" : "restricted",
+                      value: checked ? "PUBLIC" : "RESTRICTED",
                     },
                   });
                   setPublicVisibility(checked);
                 }}
-                defaultChecked={
-                  scanreport.visibility === "shared" ? true : false
-                }
+                checked={values.visibility === "PUBLIC"}
                 disabled={!canUpdate}
               />
               <Label className="text-lg">
-                {values.visibility === "shared" ? "shared" : "restricted"}
+                {/* Show user-friendly label */}
+                {values.visibility === "PUBLIC" ? "Shared" : "Restricted"}
               </Label>
             </div>
             {!publicVisibility && (