From f173f2106fc55ce5afca53c877ba14c2653ff2a7 Mon Sep 17 00:00:00 2001 From: Ben Gotow Date: Tue, 10 Dec 2024 16:18:38 -0600 Subject: [PATCH] =?UTF-8?q?[ui]=20Avoid=20a=20=E2=80=9C0=20partitions?= =?UTF-8?q?=E2=80=9D=20tag=20when=20an=20un-partitioned=20asset=20is=20bac?= =?UTF-8?q?kfilled=20(#26361)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary & Motivation This is a tiny fix for https://linear.app/dagster-labs/issue/FE-703/backfill-table-non-partitioned-assets-show-0-partitions ## How I Tested These Changes I verified this manually, but I don't think it's worth writing a test for it because this backfills page may be getting folded away into the runs feed soon. Also verified that the new runs feed doesn't have this issue. Co-authored-by: bengotow --- .../packages/ui-core/src/instance/backfill/BackfillRow.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/js_modules/dagster-ui/packages/ui-core/src/instance/backfill/BackfillRow.tsx b/js_modules/dagster-ui/packages/ui-core/src/instance/backfill/BackfillRow.tsx index 392385df56fa9..791e91320469b 100644 --- a/js_modules/dagster-ui/packages/ui-core/src/instance/backfill/BackfillRow.tsx +++ b/js_modules/dagster-ui/packages/ui-core/src/instance/backfill/BackfillRow.tsx @@ -238,7 +238,7 @@ const BackfillRequestedRange = ({ }) => { const {partitionNames, numPartitions} = backfill; - if (numPartitions === null) { + if (numPartitions === null || numPartitions === 0) { return ; }