From 5e9739c4ec831bad032a7324093d01e504a60110 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thomas=20Oberm=C3=BCller?= <thomas.obermueller@gmail.com>
Date: Wed, 31 Jul 2024 11:10:00 +0200
Subject: [PATCH] feat(notebooks): add error boundary around components
 (#24095)

---
 .../scenes/notebooks/Nodes/NodeWrapper.tsx    | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/frontend/src/scenes/notebooks/Nodes/NodeWrapper.tsx b/frontend/src/scenes/notebooks/Nodes/NodeWrapper.tsx
index 64f079536d670..c0ddb29364f2d 100644
--- a/frontend/src/scenes/notebooks/Nodes/NodeWrapper.tsx
+++ b/frontend/src/scenes/notebooks/Nodes/NodeWrapper.tsx
@@ -259,11 +259,13 @@ function NodeWrapper<T extends CustomNotebookNodeAttributes>(props: NodeWrapperP
 
                                         {Settings && editingNodeId === nodeId && containerSize === 'small' ? (
                                             <div className="NotebookNode__settings">
-                                                <Settings
-                                                    key={nodeId}
-                                                    attributes={attributes}
-                                                    updateAttributes={updateAttributes}
-                                                />
+                                                <ErrorBoundary>
+                                                    <Settings
+                                                        key={nodeId}
+                                                        attributes={attributes}
+                                                        updateAttributes={updateAttributes}
+                                                    />
+                                                </ErrorBoundary>
                                             </div>
                                         ) : null}
 
@@ -278,7 +280,12 @@ function NodeWrapper<T extends CustomNotebookNodeAttributes>(props: NodeWrapperP
                                             onClick={!expanded && expandOnClick ? () => setExpanded(true) : undefined}
                                             onMouseDown={onResizeStart}
                                         >
-                                            <Component attributes={attributes} updateAttributes={updateAttributes} />
+                                            <ErrorBoundary>
+                                                <Component
+                                                    attributes={attributes}
+                                                    updateAttributes={updateAttributes}
+                                                />
+                                            </ErrorBoundary>
                                         </div>
                                     </>
                                 )}