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

Add basic visibility table component #762

Conversation

adhityamamallan
Copy link
Member

@adhityamamallan adhityamamallan commented Dec 13, 2024

Summary

  • Add useListWorkflowsBasic hook that uses useMergedInfiniteQueries to fetch data from both ListOpen and ListClosed
  • Add DomainWorkflowsBasicTable to display basic visibility table listing
  • Add util getDateDaysBeforeToday that gets a new Date "n" days before present
  • Preload dates in DomainWorkflowsBasicFilters since ListOpen/ListClosed need a date string (also add unit tests)

Test plan

Unit tests + ran locally.

Use case with filters applied:
Screenshot 2024-12-13 at 3 11 52 PM
Screenshot 2024-12-13 at 3 11 41 PM

Use case with too many filters applied:
Screenshot 2024-12-13 at 3 11 30 PM

@adhityamamallan adhityamamallan merged commit ce67a14 into cadence-workflow:release/4.0.0 Dec 13, 2024
1 check passed
@adhityamamallan adhityamamallan deleted the basic-vis-table-final branch December 13, 2024 10:24
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.

2 participants