Page MenuHomePhabricator

[dagit] change Preparing to Not Executed
ClosedPublic

Authored by alangenfeld on Tue, Oct 13, 4:21 PM.

Details

Summary

With the change to dagster to simply not execute steps down stream of error - the "Preparing" title is not accurate once the pipeline finishes.

resolves #3087

Test Plan

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

js_modules/dagit/src/gaant/GaantStatusPanel.tsx
57

thoughts on toggling based on final pipeline state?

js_modules/dagit/src/gaant/GaantStatusPanel.tsx
57

Yeah, there's a useful distinction between "this will execute" and "this did not execute and never will".

This revision is now accepted and ready to land.Wed, Oct 14, 3:04 PM