Merge pull request #191 from vtex-apps/fix/uisettings-validation

Fix/uisettings validation
vtex-apps · Dec 19, 2024 · 7d1b4d0 · 7d1b4d0
2 parents 46c58b8 + abd2d3c
commit 7d1b4d0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,9 +6,10 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+- Fix `saveB2BSettings` mutation when `uiSettings` is absent
 
 ## [0.63.1] - 2024-12-10
-
 ### Added
 
 - Added mail notification to `createOrganizationAndCostCenterWithAdminUser` mutation

diff --git a/node/resolvers/Mutations/Settings.ts b/node/resolvers/Mutations/Settings.ts
@@ -104,9 +104,11 @@ const Settings = {
           transactionEmailSettings ??
           currentB2BSettings?.transactionEmailSettings,
         uiSettings: {
-          showModal: uiSettings.showModal,
-          clearCart: uiSettings.clearCart,
-          topBar: uiSettings.topBar ?? currentB2BSettings?.uiSettings?.topBar,
+          showModal:
+            uiSettings?.showModal ?? currentB2BSettings?.uiSettings.showModal,
+          clearCart:
+            uiSettings?.clearCart ?? currentB2BSettings?.uiSettings.clearCart,
+          topBar: uiSettings?.topBar ?? currentB2BSettings?.uiSettings?.topBar,
         },
       }