Page MenuHomePhabricator

add retry button to run page upon failure
AbandonedPublic

Authored by prha on Oct 16 2019, 12:04 AM.

Details

Reviewers
None
Summary

Depends on D1294

Test Plan

bk

Diff Detail

Repository
R1 dagster
Branch
prha/resume_retry
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

prha created this revision.Oct 16 2019, 12:04 AM
prha edited the summary of this revision. (Show Details)Oct 16 2019, 12:05 AM
prha added inline comments.
js_modules/dagit/src/runs/Run.tsx
233

Right now, the button is called "Retry".

It is extremely confusing to have two buttons called "Re-Execute" and "Retry", but put up this diff so that we can discuss and then we can adjust

prha updated this revision to Diff 5870.Oct 16 2019, 6:33 AM

rebase

prha abandoned this revision.Tue, Nov 5, 1:16 AM

abandoning in favor of D1355