From d731532fbf57d5556919b3218e3d9d8ecd053887 Mon Sep 17 00:00:00 2001
From: Ephraim Atta-Duncan <ephraimduncan68@gmail.com>
Date: Tue, 2 Jan 2024 04:58:35 +0000
Subject: [PATCH] chore: hide empty accordion for documents without date field

---
 .../primitives/document-flow/add-subject.tsx  | 24 +++++++++----------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/packages/ui/primitives/document-flow/add-subject.tsx b/packages/ui/primitives/document-flow/add-subject.tsx
index d73019732..8fef8af7b 100644
--- a/packages/ui/primitives/document-flow/add-subject.tsx
+++ b/packages/ui/primitives/document-flow/add-subject.tsx
@@ -160,14 +160,14 @@ export const AddSubjectFormPartial = ({
               </ul>
             </div>
 
-            <Accordion type="multiple" className="mt-8 border-none">
-              <AccordionItem value="advanced-options" className="border-none">
-                <AccordionTrigger className="mb-2 border-b text-left hover:no-underline">
-                  Advanced Options
-                </AccordionTrigger>
+            {hasDateField && (
+              <Accordion type="multiple" className="mt-8 border-none">
+                <AccordionItem value="advanced-options" className="border-none">
+                  <AccordionTrigger className="mb-2 border-b text-left hover:no-underline">
+                    Advanced Options
+                  </AccordionTrigger>
 
-                <AccordionContent className="text-muted-foreground -mx-1 flex max-w-prose flex-col px-1 text-sm leading-relaxed">
-                  {hasDateField && (
+                  <AccordionContent className="text-muted-foreground -mx-1 flex max-w-prose flex-col px-1 text-sm leading-relaxed">
                     <div className="mt-2 flex flex-col">
                       <Label htmlFor="date-format">
                         Date Format <span className="text-muted-foreground">(Optional)</span>
@@ -194,9 +194,7 @@ export const AddSubjectFormPartial = ({
                         )}
                       />
                     </div>
-                  )}
 
-                  {hasDateField && (
                     <div className="mt-4 flex flex-col">
                       <Label htmlFor="time-zone">
                         Time Zone <span className="text-muted-foreground">(Optional)</span>
@@ -216,10 +214,10 @@ export const AddSubjectFormPartial = ({
                         )}
                       />
                     </div>
-                  )}
-                </AccordionContent>
-              </AccordionItem>
-            </Accordion>
+                  </AccordionContent>
+                </AccordionItem>
+              </Accordion>
+            )}
           </div>
         </div>
       </DocumentFlowFormContainerContent>