From 0ed90aadf1c65fe4973e0e629bf895093fe057da Mon Sep 17 00:00:00 2001
From: Patrick Segura <patrick.segura@oddball.io>
Date: Thu, 29 Feb 2024 13:31:50 -0600
Subject: [PATCH] Fixed previous names render to now render in bullet

---
 .../__snapshots__/index.test.tsx.snap         | 40 ++++++++++++++-----
 .../__snapshots__/index.test.tsx.snap         | 19 +++++++--
 .../__snapshots__/index.test.tsx.snap         | 19 +++++++--
 .../_components/ReadOnlySection/index.tsx     |  3 +-
 .../_components/ReadOnlySection/util.tsx      |  4 +-
 5 files changed, 65 insertions(+), 20 deletions(-)

diff --git a/src/components/ShareExport/__snapshots__/index.test.tsx.snap b/src/components/ShareExport/__snapshots__/index.test.tsx.snap
index affa82e623..fcd044e5a6 100644
--- a/src/components/ShareExport/__snapshots__/index.test.tsx.snap
+++ b/src/components/ShareExport/__snapshots__/index.test.tsx.snap
@@ -207,11 +207,22 @@ exports[`ShareExportModal > matches the snapshot 1`] = `
                 >
                   Previous names
                 </p>
-                <div
-                  class="margin-y-0 font-body-md line-height-sans-4 text-pre-line"
+                <ul
+                  class="margin-y-0 padding-left-3"
                 >
-                  First NameSecond Name
-                </div>
+                  <li
+                    class="font-sans-md line-height-sans-4"
+                  >
+                    First Name
+                  </li>
+                  <ul />
+                  <li
+                    class="font-sans-md line-height-sans-4"
+                  >
+                    Second Name
+                  </li>
+                  <ul />
+                </ul>
               </div>
             </div>
             <div
@@ -8240,11 +8251,22 @@ exports[`ShareExportModal > matches the snapshot 2`] = `
                 >
                   Previous names
                 </p>
-                <div
-                  class="margin-y-0 font-body-md line-height-sans-4 text-pre-line"
-                >
-                  First NameSecond Name
-                </div>
+                <ul
+                  class="margin-y-0 padding-left-3"
+                >
+                  <li
+                    class="font-sans-md line-height-sans-4"
+                  >
+                    First Name
+                  </li>
+                  <ul />
+                  <li
+                    class="font-sans-md line-height-sans-4"
+                  >
+                    Second Name
+                  </li>
+                  <ul />
+                </ul>
               </div>
             </div>
             <div
diff --git a/src/views/ModelPlan/ReadOnly/ModelBasics/__snapshots__/index.test.tsx.snap b/src/views/ModelPlan/ReadOnly/ModelBasics/__snapshots__/index.test.tsx.snap
index ee375e9012..163f42d062 100644
--- a/src/views/ModelPlan/ReadOnly/ModelBasics/__snapshots__/index.test.tsx.snap
+++ b/src/views/ModelPlan/ReadOnly/ModelBasics/__snapshots__/index.test.tsx.snap
@@ -37,11 +37,22 @@ exports[`Read Only Model Plan Summary -- Model Basics > matches snapshot 1`] = `
         >
           Previous names
         </p>
-        <div
-          class="margin-y-0 font-body-md line-height-sans-4 text-pre-line"
+        <ul
+          class="margin-y-0 padding-left-3"
         >
-          First NameSecond Name
-        </div>
+          <li
+            class="font-sans-md line-height-sans-4"
+          >
+            First Name
+          </li>
+          <ul />
+          <li
+            class="font-sans-md line-height-sans-4"
+          >
+            Second Name
+          </li>
+          <ul />
+        </ul>
       </div>
     </div>
     <div
diff --git a/src/views/ModelPlan/ReadOnly/__snapshots__/index.test.tsx.snap b/src/views/ModelPlan/ReadOnly/__snapshots__/index.test.tsx.snap
index ae701faccd..575fd27603 100644
--- a/src/views/ModelPlan/ReadOnly/__snapshots__/index.test.tsx.snap
+++ b/src/views/ModelPlan/ReadOnly/__snapshots__/index.test.tsx.snap
@@ -530,11 +530,22 @@ exports[`Read Only Model Plan Summary > matches snapshot 1`] = `
                           >
                             Previous names
                           </p>
-                          <div
-                            class="margin-y-0 font-body-md line-height-sans-4 text-pre-line"
+                          <ul
+                            class="margin-y-0 padding-left-3"
                           >
-                            First NameSecond Name
-                          </div>
+                            <li
+                              class="font-sans-md line-height-sans-4"
+                            >
+                              First Name
+                            </li>
+                            <ul />
+                            <li
+                              class="font-sans-md line-height-sans-4"
+                            >
+                              Second Name
+                            </li>
+                            <ul />
+                          </ul>
                         </div>
                       </div>
                       <div
diff --git a/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/index.tsx b/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/index.tsx
index 485ce72d94..8492ac8232 100644
--- a/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/index.tsx
+++ b/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/index.tsx
@@ -141,7 +141,8 @@ const RenderReadonlyValue = <
   // Renders a single value
   if (
     isTranslationFieldProperties(config) &&
-    !isTranslationFieldPropertiesWithOptions(config)
+    !isTranslationFieldPropertiesWithOptions(config) &&
+    !config.isArray
   ) {
     return <SingleValue value={value} />;
   }
diff --git a/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/util.tsx b/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/util.tsx
index 8c12cc2d4f..7a636efde8 100644
--- a/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/util.tsx
+++ b/src/views/ModelPlan/ReadOnly/_components/ReadOnlySection/util.tsx
@@ -28,9 +28,9 @@ export const formatListValues = <
     | TranslationFieldPropertiesWithOptions<T>, // Translation config
   value: T[] | undefined // field value/enum array
 ): string[] => {
-  if (!isTranslationFieldPropertiesWithOptions(config)) return [];
+  if (config.isArray || config.isModelLinks) return value as string[];
 
-  if (config.isModelLinks) return value as string[];
+  if (!isTranslationFieldPropertiesWithOptions(config)) return [];
 
   return getKeys(config.options)
     .filter(option => Array.isArray(value) && value?.includes(option))