From 91f0e01d0f377e7b39ce8b1711db59a2e11ccc0e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Frane=20Poli=C4=87?=
 <16856471+fPolic@users.noreply.github.com>
Date: Tue, 28 Mar 2023 14:56:07 +0200
Subject: [PATCH] fix(admin-ui): disallow creating OE if there is no changes
 (#3604)

* wip: fix modal

* refactor: revert log

---------

Co-authored-by: fPolic <frane@medusajs.com>
---
 packages/admin-ui/ui/src/domain/orders/edit/modal.tsx | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/admin-ui/ui/src/domain/orders/edit/modal.tsx b/packages/admin-ui/ui/src/domain/orders/edit/modal.tsx
index a038f6eeb9..ac69c76e93 100644
--- a/packages/admin-ui/ui/src/domain/orders/edit/modal.tsx
+++ b/packages/admin-ui/ui/src/domain/orders/edit/modal.tsx
@@ -167,7 +167,7 @@ function OrderEditModal(props: OrderEditModalProps) {
   const [filterTerm, setFilterTerm] = useState<string>("")
 
   const showTotals = currentSubtotal !== orderEdit.subtotal
-  const showNote = !!orderEdit.changes.length
+  const hasChanges = !!orderEdit.changes.length
 
   const {
     mutateAsync: requestConfirmation,
@@ -344,7 +344,7 @@ function OrderEditModal(props: OrderEditModalProps) {
           )}
 
           {/* NOTE */}
-          {showNote && (
+          {hasChanges && (
             <div className="flex items-center justify-between">
               <span className="text-gray-500">Note</span>
               <InputField
@@ -370,7 +370,7 @@ function OrderEditModal(props: OrderEditModalProps) {
               variant="primary"
               size="small"
               type="button"
-              disabled={isUpdating || isRequestingConfirmation}
+              disabled={isUpdating || isRequestingConfirmation || !hasChanges}
               loading={isUpdating || isRequestingConfirmation}
               onClick={onSave}
             >