Skip to content

Commit

Permalink
[FE-769] Show other automations if an asset has an automation conditi…
Browse files Browse the repository at this point in the history
…on (#28123)

## Summary & Motivation

As titled.

## How I Tested These Changes

before:
<img width="260" alt="Screenshot 2025-02-28 at 10 03 43 AM"
src="https://github.com/user-attachments/assets/2c8d97a0-d6ea-4014-8554-f0e2dfb113d5"
/>

after:
<img width="306" alt="Screenshot 2025-02-28 at 10 03 51 AM"
src="https://github.com/user-attachments/assets/9f31d5fc-302d-45e0-ad6b-4db9b8f2d0bd"
/>


## Changelog

[ui] Fixed an issue where assets with automation conditions wouldn't
show the jobs/sensors/schedules targeting them.
  • Loading branch information
salazarm authored Feb 28, 2025
1 parent ef0670e commit 9ad4643
Showing 1 changed file with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,15 @@ export const AutomationDetailsSection = ({
<SectionSkeleton />
),
},
{
label: 'Automation condition',
children: assetNode?.automationCondition?.label ? (
<EvaluationUserLabel
userLabel={assetNode?.automationCondition.label}
expandedLabel={assetNode?.automationCondition.expandedLabel}
/>
) : null,
},
];

if (
Expand All @@ -85,15 +94,6 @@ export const AutomationDetailsSection = ({
learnMoreLink="https://docs.dagster.io/concepts/automation#automation"
/>
);
} else {
if (assetNode?.automationCondition && assetNode?.automationCondition.label) {
return (
<EvaluationUserLabel
userLabel={assetNode?.automationCondition.label}
expandedLabel={assetNode?.automationCondition.expandedLabel}
/>
);
}
}

return (
Expand Down

1 comment on commit 9ad4643

@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-2vumwe7yh-elementl.vercel.app

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

Please sign in to comment.