Page MenuHomeElementl

Merge scheduler page and schedules page

Authored by dgibson on Nov 13 2020, 8:37 PM.



This diff makes the scheduler page and the schedules-for-a-repository page use the same UI. The only difference is that the scheduler page is not repository-scoped.

Both pages show a list of schedules and a list of unloadable schedules. The scheduler page groups the schedules by repository.

In particular there are no longer any prompts to reconcile. The only reconcile-like experience you have now is the 'delete a now-unloadable schedule' experience.

Test Plan

Load a repo with two repositories, visit scheduler page and schedules page for each repository
Turn some schedules on and off, leave some on
Remove a running schedule from the codebase
Visit scheduler page and schedules page for both repos again, verify you can turn off the no longer existant schedule
Load dagit with empty workspace, see all running schedules available on scheduler page to turn off

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

dgibson edited the summary of this revision. (Show Details)
dgibson edited the test plan for this revision. (Show Details)
dgibson added reviewers: sashank, dish, alangenfeld, prha.
dgibson added a subscriber: schrockn.

up, ready for review

This looks great! Awesome to see that reconcile UI go away :)


I think this should never be true, since we're not passing in a repository selector argument to this field. Still, it's probably good to check for it, but we might want to show a different invalid state than "Repository not found" because it doesn't really make sense here. It's more like "something went wrong in the Dagster framework itself"


Nit/Tip: You can alias this name right here in the GQL query instead of needing to do it in JS, so that the field name here makes a bit more sense:

unLoadableScheduleStates: scheduleStatesOrError(withNoScheduleDefinition: true) { ... }

Then above you can do:

const {
} = result;

Nit: Extra space

This revision is now accepted and ready to land.Nov 13 2020, 11:29 PM
This revision was automatically updated to reflect the committed changes.