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
Branch
prha/remove_schedule_config
Lint
Lint OK
Unit
No Unit Test Coverage

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
1497–1499 ↗(On Diff #26087)

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

rip out ScheduleExecutionDataMode