Skip to content

Workflow Relationship Tree#2487

Merged
Alex-Tideman merged 72 commits intomainfrom
workflow-family-tree
Jan 9, 2025
Merged

Workflow Relationship Tree#2487
Alex-Tideman merged 72 commits intomainfrom
workflow-family-tree

Conversation

@Alex-Tideman
Copy link
Copy Markdown
Collaborator

@Alex-Tideman Alex-Tideman commented Jan 7, 2025

Description & motivation 💭

Allows user to view the full relationship tree of a workflow, not just direct children. Fetches root workflow and renders tree of all relationships. Includes showing Start/End Times and Duration as well as Type, ID of each Workflow. If rootWorkflow fetch fails or is not supported, fallback to previous tables.

Also includes tree view for CaN and Scheduled workflows, and small responsive padding improvements to the Workflow pages.

Screenshots (if applicable) 📸

Screenshot 2025-01-08 at 3 22 00 PM Screenshot 2025-01-08 at 3 23 08 PM Screenshot 2025-01-08 at 3 22 07 PM Screenshot 2025-01-08 at 3 22 24 PM Screenshot 2025-01-08 at 3 22 37 PM

Design Considerations 🎨

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

Docs

Any docs updates needed?

laurakwhit and others added 30 commits September 12, 2024 14:52
* Fix styling for non-expandable Accordion

* Update padding on Card to match Accordion

* Add AccordionGroup

* Make ring inset on focus-within

* Fix imports and A11y warning
* Add active styling for nav item

* Add isActive check for nav links

* Account for if isActive is undefined
* Use AccordionGroup to show all possible failures

* Dont use red text

* Move source to accordion

* Add pending task component and move into accordion group

* Fixes from Laura's review
Bumps [express](https://github.com/expressjs/express) from 4.19.2 to 4.20.0.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.19.2...4.20.0)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.0.13 to 5.2.14.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v5.2.14/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.2.14/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update go to 1.23

* Use 1.22.6

* Remove work files

* Add go.work to gitignore
* Remove unnecessary coerced Boolean

* Pluralize Namespaces badge for Nexus endpoints

* Fix action slot on Accordion component

* Ignore a11y test on Accordion with action
Co-authored-by: Temporal Data (cicd) <commander-data@temporal.io>
* Update go version to 1.23 in update ui-server

* Use v4
@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
holocene ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 8, 2025 9:42pm

@Alex-Tideman Alex-Tideman merged commit f978005 into main Jan 9, 2025
@Alex-Tideman Alex-Tideman deleted the workflow-family-tree branch January 9, 2025 17:45
hferentschik pushed a commit to DataDog/temporalio-ui that referenced this pull request Mar 12, 2026
* Accordion component updates (temporalio#2316)

* Fix styling for non-expandable Accordion

* Update padding on Card to match Accordion

* Add AccordionGroup

* Make ring inset on focus-within

* Fix imports and A11y warning

* Highlight active link in nav bar (temporalio#2324)

* Add active styling for nav item

* Add isActive check for nav links

* Account for if isActive is undefined

* Add workflow electron

* Add http api port and async update flag (temporalio#2314)

* Show all possible failures for WorkflowTaskError (temporalio#2328)

* Use AccordionGroup to show all possible failures

* Dont use red text

* Move source to accordion

* Add pending task component and move into accordion group

* Fixes from Laura's review

* Add border

* WIP: filter

* Bump express from 4.19.2 to 4.20.0 (temporalio#2323)

Bumps [express](https://github.com/expressjs/express) from 4.19.2 to 4.20.0.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.19.2...4.20.0)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump vite from 5.0.13 to 5.2.14 (temporalio#2329)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.0.13 to 5.2.14.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v5.2.14/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.2.14/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix the go things

* MVP of electron. Needs some performance tweaks and to add links to each node

* Add icons, clean up

* Remove commented out code

* Update snapshots

* Update go to 1.22.6 (temporalio#2334)

* Update go to 1.23

* Use 1.22.6

* Remove work files

* Add go.work to gitignore

* Small UI fixes (temporalio#2345)

* Remove unnecessary coerced Boolean

* Pluralize Namespaces badge for Nexus endpoints

* Fix action slot on Accordion component

* Ignore a11y test on Accordion with action

* 2.31.1 (temporalio#2346)

Co-authored-by: Temporal Data (cicd) <commander-data@temporal.io>

* Update version.go to v2.31.0 (temporalio#2347)

* Use go 1.23.0 (temporalio#2348)

* Use 1.23 (temporalio#2349)

* Update go version to 1.23 in update ui-server (temporalio#2351)

* Update go version to 1.23 in update ui-server

* Use v4

* Add breadcrumbs for nodes

* Add hover/click state with opacity

* Add back child table

* Fix overfetching issue

* Fix icon type

* WIP Family Tree

* Get math right for tree positions

* I did things but can't remember what

* Remove electron stuff

* Get rid of pane

* Add side panel descriptions

* Better styles and status color

* Better side nav buttons. Better lines to dots. Need to fix the tree node expanding

* Add link, needs a better place

* Better styles on description, add links, fix opening nodes

* Add table of info and link vis/sidenav with open/close

* Add table

* Better description

* fix lines

* Better isActive, isCurrent checks

* isCurrent check

* Make split 50/50

* Fix types

* Better styles for active, better borders.

* Refactor to include root, clean up styles, better basis widths

* Add root child count on node tree

* Make tree sticky on lg screens

* Add CaN tree

* Add schedule tree

* Delete all unused tables

* Delete breadcrumbs

* Use justify-end to move links to far right

* Fix sm screen layout

* Make visual a little shorter at 240px, hide times if small screen

* Add dash and min-w for end time, py-1

* Fix root node click and accessibility issues

* Bump maxZoomOut for a lot of children

* Add back old tables in case api not supported or if we wanna go labs mode route

* Small style updates based on feedback, better responsive padding

* Better count placement

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Laura Whitaker <laura.whitaker@temporal.io>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: temporal-cicd[bot] <138905749+temporal-cicd[bot]@users.noreply.github.com>
Co-authored-by: Temporal Data (cicd) <commander-data@temporal.io>
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.

3 participants