Skip to content

Commit

Permalink
[ui] Avoid a “0 partitions” tag when an un-partitioned asset is backf…
Browse files Browse the repository at this point in the history
…illed (#26361)

## 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 <[email protected]>
  • Loading branch information
bengotow and bengotow authored Dec 10, 2024
1 parent 3032429 commit f173f21
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ const BackfillRequestedRange = ({
}) => {
const {partitionNames, numPartitions} = backfill;

if (numPartitions === null) {
if (numPartitions === null || numPartitions === 0) {
return <span />;
}

Expand Down

1 comment on commit f173f21

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-a68qo0j39-elementl.vercel.app

Built with commit f173f21.
This pull request is being automatically deployed with vercel-action

Please sign in to comment.