From 0daf6840925286b90a430987f886dacc28b0ad4a Mon Sep 17 00:00:00 2001
From: philippe <t4rk@outlook.com>
Date: Mon, 24 Feb 2025 10:21:24 -0500
Subject: [PATCH 1/4] Fix missing id from ExternalWrapper.

---
 dash/dash-renderer/src/wrapper/ExternalWrapper.tsx    | 4 +++-
 tests/integration/renderer/test_external_component.py | 6 ++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx b/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx
index 79bcbb5162..e8c2c1f476 100644
--- a/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx
+++ b/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx
@@ -51,7 +51,9 @@ function ExternalWrapper({
     useEffect(() => {
         batch(() => {
             dispatch(updateProps({itempath: componentPath, props}));
-            dispatch(notifyObservers({id: props.id, props}));
+            if (props.id) {
+                dispatch(notifyObservers({id: props.id, props}));
+            }
         });
     }, [props]);
 
diff --git a/tests/integration/renderer/test_external_component.py b/tests/integration/renderer/test_external_component.py
index 72657125f5..c0ab76fcc3 100644
--- a/tests/integration/renderer/test_external_component.py
+++ b/tests/integration/renderer/test_external_component.py
@@ -23,6 +23,10 @@ def test_rext001_render_external_component(dash_duo):
                     },
                 },
             ),
+            ExternalComponent(
+                id="without-id",
+                text="without-id",
+            ),
             html.Div(html.Div(id={"type": "output", "index": 1}), id="out"),
         ]
     )
@@ -53,3 +57,5 @@ def click(*_):
 
     dash_duo.find_element("#extra > div").click()
     dash_duo.wait_for_text_to_equal("#out", "clicked")
+
+    assert dash_duo.get_logs() == []

From 10a6d973a1ede63dbcbe6a53d11de6a4f98d4d5f Mon Sep 17 00:00:00 2001
From: philippe <t4rk@outlook.com>
Date: Mon, 24 Feb 2025 10:57:18 -0500
Subject: [PATCH 2/4] Refactor ExternalWrapper props

---
 .../src/components/ExternalComponent.js       | 21 +++++++-----
 .../src/wrapper/ExternalWrapper.tsx           | 34 ++++++++-----------
 2 files changed, 27 insertions(+), 28 deletions(-)

diff --git a/@plotly/dash-test-components/src/components/ExternalComponent.js b/@plotly/dash-test-components/src/components/ExternalComponent.js
index d82fbd93a0..64d2efc515 100644
--- a/@plotly/dash-test-components/src/components/ExternalComponent.js
+++ b/@plotly/dash-test-components/src/components/ExternalComponent.js
@@ -8,20 +8,23 @@ const ExternalComponent = ({ id, text, input_id, extra_component }) => {
 
     return (
         <div id={id}>
-            <ExternalWrapper
-                id={input_id}
-                componentType="Input"
-                componentNamespace="dash_core_components"
-                value={text}
+            {text && <ExternalWrapper
+                
+                component={{
+                    type: "Input",
+                    namespace: "dash_core_components",
+                    props: {
+                        value: text,
+                        id: input_id
+                    }
+                }}
                 componentPath={[...ctx.componentPath, 'external']}
-            />
+            />}
             {
                 extra_component &&
                 <ExternalWrapper
-                    componentType={extra_component.type}
-                    componentNamespace={extra_component.namespace}
+                    component={extra_component}
                     componentPath={[...ctx.componentPath, 'extra']}
-                    {...extra_component.props}
             />}
         </div>
     )
diff --git a/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx b/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx
index e8c2c1f476..a6b1dad13f 100644
--- a/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx
+++ b/dash/dash-renderer/src/wrapper/ExternalWrapper.tsx
@@ -1,7 +1,7 @@
 import React, {useState, useEffect} from 'react';
 import {batch, useDispatch} from 'react-redux';
 
-import {DashLayoutPath} from '../types/component';
+import {DashComponent, DashLayoutPath} from '../types/component';
 import DashWrapper from './DashWrapper';
 import {
     addComponentToLayout,
@@ -11,21 +11,14 @@ import {
 } from '../actions';
 
 type Props = {
+    component: DashComponent;
     componentPath: DashLayoutPath;
-    componentType: string;
-    componentNamespace: string;
-    [k: string]: any;
 };
 
 /**
  * For rendering components that are out of the regular layout tree.
  */
-function ExternalWrapper({
-    componentType,
-    componentNamespace,
-    componentPath,
-    ...props
-}: Props) {
+function ExternalWrapper({component, componentPath}: Props) {
     const dispatch: any = useDispatch();
     const [inserted, setInserted] = useState(false);
 
@@ -34,11 +27,7 @@ function ExternalWrapper({
         // The props will come from the parent so they can be updated.
         dispatch(
             addComponentToLayout({
-                component: {
-                    type: componentType,
-                    namespace: componentNamespace,
-                    props: props
-                },
+                component,
                 componentPath
             })
         );
@@ -50,12 +39,19 @@ function ExternalWrapper({
 
     useEffect(() => {
         batch(() => {
-            dispatch(updateProps({itempath: componentPath, props}));
-            if (props.id) {
-                dispatch(notifyObservers({id: props.id, props}));
+            dispatch(
+                updateProps({itempath: componentPath, props: component.props})
+            );
+            if (component.props.id) {
+                dispatch(
+                    notifyObservers({
+                        id: component.props.id,
+                        props: component.props
+                    })
+                );
             }
         });
-    }, [props]);
+    }, [component.props]);
 
     if (!inserted) {
         return null;

From 944187354bc61dc6e7c846db38dadfa4a08f83cd Mon Sep 17 00:00:00 2001
From: philippe <t4rk@outlook.com>
Date: Mon, 24 Feb 2025 11:16:18 -0500
Subject: [PATCH 3/4] build


From 6cf16b5d0ee07786e66f22f9de8e29dd6c0bf02a Mon Sep 17 00:00:00 2001
From: philippe <t4rk@outlook.com>
Date: Mon, 24 Feb 2025 11:34:28 -0500
Subject: [PATCH 4/4] build