feat(schedule-details): add metrics chart run hover popovers (SLICE-9.3 / PR09f)#21
Open
Assem-Uber wants to merge 5 commits into
Conversation
…PR09d) Render visx markers for successful runs, missed executions, and the next execution time from static fixture data, replacing the empty state when series data is present. Co-authored-by: Cursor <cursoragent@cursor.com>
Introduce an infinite-query hook that lists active workflows for a schedule via advanced visibility (CadenceScheduleID predicate, CadenceScheduleTime sort) so the metrics chart can paginate execution data in PR09e. Co-authored-by: Cursor <cursoragent@cursor.com>
… (SLICE-9.2 / PR09e) Replace static chart fixtures with useListWorkflowsForSchedule and useDescribeSchedule, map workflow pages to timeline points, pan/scroll into older time to fetchNextPage, and show loading skeletons while data loads. Co-authored-by: Cursor <cursoragent@cursor.com>
….3 / PR09f) Add interactive glyph popovers on the Runs metrics chart to inspect scheduled runs without extra API calls, including run IDs, status with icons, optional backfill links, and run timestamps. This also adds hover/focus behavior and coverage for stacked multi-run glyphs. Co-authored-by: Cursor <cursoragent@cursor.com>
Apply eslint import-order and prettier fixes across the metrics chart slice so CI lint passes for PR09f hover popover work. Co-authored-by: Cursor <cursoragent@cursor.com>
91ac5f9 to
2aa2fba
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
StatefulPopoverScheduleMetricsChartRun, group stacked runs by scheduled time, and add browser/node testsNotes
feat/schedule-details-pr09e-live-workflows(PR09e / SLICE-9.2)feat/schedule-details-pr09e-live-workflowsonce PR09e merges to cadence-workflow/cadence-webTest plan
npm run test:unit:browser -- schedule-detail-metrics-chartnpm run test:unit:node -- workflows-for-schedule-to-chart-pointsMade with Cursor