Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(dashboard): display scheduledWfRuns #1202

Merged
merged 15 commits into from
Dec 19, 2024
Merged

Conversation

bryson-g
Copy link
Contributor

@bryson-g bryson-g commented Dec 17, 2024

Resolves #1087

Description

  • Tab design between WfSpecs & ScheduledWfSpecs.
  • SelectionLink component used for WfSpec list & ScheduledWfSpec for consistent design.
  • Stateful loading pattern for ScheduledWfSpecs, since proto buff does not support queries. -SearchScheduledWfRunRequest should support querying for pagination #1201
  • Filter ScheduledWfSpecs by time window for next expected run datetime.

Considerations

  • We may want another approach for filtering ScheduledWfRuns, such as by the cron expression categories themselves.
  • More relevant data may want to be displayed for each listed ScheduledWfRun, in which case a table UI would be good:

https://ui.shadcn.com/docs/components/data-table
image

Videos

Before

Screen.Recording.2024-12-16.at.3.56.01.PM.mov

After

Screen.Recording.2024-12-16.at.4.03.49.PM.mov

@bryson-g bryson-g requested a review from hazimoarafa December 17, 2024 02:21
@hazimoarafa
Copy link
Member

hazimoarafa commented Dec 17, 2024

I think a table would be great idea not only for scheduled Wfs but just normal WfRuns

@bryson-g bryson-g requested a review from hazimoarafa December 17, 2024 02:37
@hazimoarafa hazimoarafa removed their request for review December 17, 2024 02:48
@hazimoarafa
Copy link
Member

Looks like lh-standalone broke, please fix.

@bryson-g bryson-g requested a review from hazimoarafa December 18, 2024 02:41
@bryson-g bryson-g merged commit 3807fad into master Dec 19, 2024
13 checks passed
@bryson-g bryson-g deleted the dashboard-scheduledwfruns branch December 19, 2024 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show ScheduledWfRuns on the Dashboard
2 participants