Page MenuHomeElementl

[dagit] Fix sensor/schedule mutations
ClosedPublic

Authored by dish on May 10 2021, 10:59 PM.

Details

Summary

TypeScript allowed me to have name and status vars that didn't pull values from the GraphQL payload, so the sensor and schedule switches broke with https://dagster.phacility.com/D7839. This fixes the vars.

Test Plan

View sensor and schedule in Dagit. Turn them on/off, verify success and correct switch state after action.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dish published this revision for review.May 10 2021, 10:59 PM

thx for the quick fix ๐Ÿ™

This revision is now accepted and ready to land.May 10 2021, 11:00 PM
prha added a subscriber: prha.

should we do a hotfix release for this?

I just broke this a couple hours ago, so I don't think so...

This revision was automatically updated to reflect the committed changes.