fix: BugWithReusingSpace (#100)

* feat: include id from props not config * chore: changeset * Update .changeset/funny-shirts-argue.md Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
FlatFilers · Apr 26, 2024 · 1473f79 · 1473f79
1 parent 8e3949f
commit 1473f79
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/.changeset/funny-shirts-argue.md b/.changeset/funny-shirts-argue.md
@@ -0,0 +1,5 @@
+---
+'@flatfile/react': patch
+---
+
+Fixes bug with re-using spaces.
diff --git a/apps/react/app/re-used-space/App.tsx b/apps/react/app/re-used-space/App.tsx
@@ -93,8 +93,8 @@ const App = () => {
         </button>
       </div>
       <Space
+        id={SPACE_ID}
         config={{
-          id: SPACE_ID,
           metadata: {
             sidebarConfig: {
               showSidebar: true,

diff --git a/packages/react/src/components/Space.tsx b/packages/react/src/components/Space.tsx
@@ -5,7 +5,7 @@ import { useDeepCompareEffect } from '../utils/useDeepCompareEffect'
 
 type SpaceProps = {
   id?: string
-  config: Flatfile.SpaceConfig
+  config?: Flatfile.SpaceConfig
   children?: React.ReactNode
 }
 
@@ -33,11 +33,14 @@ type SpaceProps = {
  */
 
 export const Space = (props: SpaceProps) => {
-  const { config, children } = props
+  const { config, id, children } = props
   const { updateSpace } = useContext(FlatfileContext)
 
   const callback = useCallback(() => {
-    updateSpace(config)
+    updateSpace({
+      ...(config || {}),
+      id,
+    })
   }, [config, updateSpace])
 
   useDeepCompareEffect(callback, [config])