fix: only render delete button if owner (#6837)

* fix: only render delete button if owner * fix: include divider in the conditional render
opengovsg · Oct 31, 2023 · 1760b2c · 1760b2c
1 parent 36c4761
commit 1760b2c
Showing 1 changed file with 14 additions and 9 deletions.
diff --git a/frontend/src/features/workspace/components/WorkspaceFormRow/RowActions/RowActionsDrawer.tsx b/frontend/src/features/workspace/components/WorkspaceFormRow/RowActions/RowActionsDrawer.tsx
@@ -53,6 +53,7 @@ export const RowActionsDrawer = ({
     previewFormLink,
     handleDeleteForm,
     handleDuplicateForm,
+    isFormAdmin,
     handleCollaborators,
     handleShareForm,
   } = useRowAction(formMeta)
@@ -156,15 +157,19 @@ export const RowActionsDrawer = ({
                       <BiChevronRight fontSize="1.25rem" />
                     </Flex>
                   </Button>
-                  <Divider />
-                  <Button
-                    {...buttonProps}
-                    onClick={handleDeleteForm}
-                    color="danger.500"
-                    leftIcon={<BiTrash fontSize="1.25rem" />}
-                  >
-                    Delete
-                  </Button>
+                  {isFormAdmin && (
+                    <>
+                      <Divider />
+                      <Button
+                        {...buttonProps}
+                        onClick={handleDeleteForm}
+                        color="danger.500"
+                        leftIcon={<BiTrash fontSize="1.25rem" />}
+                      >
+                        Delete
+                      </Button>
+                    </>
+                  )}
                 </>
               )}
             </ButtonGroup>