From 3be740c019650ec1fe0603a234f192bd1c274ca6 Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Fri, 8 May 2026 00:02:11 +1000
Subject: [PATCH] [UI] Fix child builds panel (#11886)

* [UI] Fix child builds panel

- Adjust logic for displaying or hiding child build panel

Co-authored-by: Copilot <copilot@github.com>

* Add brief playwright test

Co-authored-by: Copilot <copilot@github.com>

---------

Co-authored-by: Copilot <copilot@github.com>
---
 src/frontend/src/pages/build/BuildDetail.tsx | 25 ++++++++++++++++++--
 src/frontend/tests/pages/pui_build.spec.ts   |  6 +++++
 2 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/src/frontend/src/pages/build/BuildDetail.tsx b/src/frontend/src/pages/build/BuildDetail.tsx
index eb3b84117e..6f4f2ac253 100644
--- a/src/frontend/src/pages/build/BuildDetail.tsx
+++ b/src/frontend/src/pages/build/BuildDetail.tsx
@@ -197,6 +197,27 @@ export default function BuildDetail() {
     defaultValue: {}
   });
 
+  // Fetch the number of child build orders associated with this build order
+  const { instance: childBuildData } = useInstance({
+    endpoint: ApiEndpoints.build_order_list,
+    params: {
+      parent: id,
+      limit: 1
+    },
+    disabled: !id,
+    hasPrimaryKey: false,
+    defaultValue: {}
+  });
+
+  /**
+   * Display the "Child Build Orders" panel if either:
+   * - There are any child build orders (childBuildData.count > 0)
+   * - There are any sub-assembly items (subassemblyLineData.count > 0)
+   */
+  const showChildBuilds = useMemo(() => {
+    return childBuildData?.count > 0 || subassemblyLineData?.count > 0;
+  }, [childBuildData, subassemblyLineData]);
+
   const buildStatus = useStatusCodes({ modelType: ModelType.build });
 
   const {
@@ -540,7 +561,7 @@ export default function BuildDetail() {
         name: 'child-orders',
         label: t`Child Build Orders`,
         icon: <IconSitemap />,
-        hidden: (subassemblyLineData?.count ?? 0) <= 0, // Hide if no sub-assembly items
+        hidden: !showChildBuilds,
         content: build.pk ? (
           <BuildOrderTable parentBuildId={build.pk} />
         ) : (
@@ -578,7 +599,7 @@ export default function BuildDetail() {
     user,
     buildStatus,
     globalSettings,
-    subassemblyLineData,
+    showChildBuilds,
     buildLineQuery.isFetching,
     buildLineQuery.isLoading,
     buildLineData
diff --git a/src/frontend/tests/pages/pui_build.spec.ts b/src/frontend/tests/pages/pui_build.spec.ts
index 7f4e8fde81..0e01e57c99 100644
--- a/src/frontend/tests/pages/pui_build.spec.ts
+++ b/src/frontend/tests/pages/pui_build.spec.ts
@@ -437,6 +437,12 @@ test('Build Order - Auto Allocate Tracked', async ({ browser }) => {
     url: 'manufacturing/build-order/27/consumed-stock'
   });
 
+  // Check some tabs along the way
+  await loadTab(page, 'Child Build Orders');
+  await page
+    .getByRole('button', { name: 'action-button-add-build-order' })
+    .waitFor();
+
   await loadTab(page, 'Incomplete Outputs');
 
   await page.getByRole('cell', { name: '0 / 6' }).waitFor();