Page MenuHomePhabricator

Remove graphql support for schedule-level run config
ClosedPublic

Authored by prha on Fri, Nov 20, 7:31 PM.

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

Harbormaster returned this revision to the author for changes because remote builds failed.Fri, Nov 20, 7:46 PM
Harbormaster failed remote builds in B21498: Diff 26086!
prha requested review of this revision.Fri, Nov 20, 8:00 PM

sweet. we should also remove ScheduleExecutionDataMode.PREVIEW now I think?

This revision is now accepted and ready to land.Fri, Nov 20, 11:00 PM
js_modules/dagit/src/schema.graphql
1496–1498

hm how did this type survive the purge? Do we still use it somewhere?

rip out ScheduleExecutionDataMode