Page MenuHomeElementl

(config-scaffold 1/n) Fix config errors visualization in Dagit
ClosedPublic

Authored by sashank on Oct 18 2020, 4:53 AM.

Details

Summary

When a solid was missing an input or config in the Playground, we incorrectly displayed that the solid had valid configuration in the "error preview" section. This diff updates the error logic to display the "invalid" state when any subconfiguration for a node is missing.

Test Plan

Fill out solid config, delete one part of the solid config that was required, verify that the solid box showed an error.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sashank added reviewers: bengotow, dish, schrockn.
sashank edited the test plan for this revision. (Show Details)

clean

This looks great! Good catch 👍

This revision is now accepted and ready to land.Oct 19 2020, 9:10 PM