Open
Conversation
dennis-zyska
requested changes
Feb 13, 2026
Collaborator
dennis-zyska
left a comment
There was a problem hiding this comment.
Nice job. There is one feature that could be interesting to implement: Giving the usual users the possibility to add additional user-based workflows and/or also to give them the opportunity to display existing workflows in the graph. The rest of the comments are mainly cleaning the code a bit.
frontend/src/components/dashboard/workflows/WorkflowEditModal.vue
Outdated
Show resolved
Hide resolved
frontend/src/components/dashboard/workflows/WorkflowRenameModal.vue
Outdated
Show resolved
Hide resolved
frontend/src/components/dashboard/workflows/ImportFormatModal.vue
Outdated
Show resolved
Hide resolved
frontend/src/components/dashboard/workflows/ExportFormatModal.vue
Outdated
Show resolved
Hide resolved
dennis-zyska
requested changes
Mar 2, 2026
Collaborator
There was a problem hiding this comment.
Very nice integration, here are some open points I found when testing the workflow integration:
- As a guest, I can't see the Workflows in the navigation
- We should also show the last updated timestamp in the overview table and the number of steps
- When rename the workflow, the button text is "Create New Workflow" instead of "Rename workflow"
- It would be good to also export a single workflow, e.g. with an additional export button in the workflow list.
- In the workflow navigation (where I add/delete new steps), when I hover over an action button, there should be a tooltip for the explanation what that buttons do
- As a user I can see all Workflows in readonly (which is good), but I can't create a copy of an existing one
- Also I can't copy a specific step from an existing one or see any information of the steps (like the configuration json etc.)
- As a user it is also not clear to me, why I can't do all those things, maybe we should add an additional Flag (like System Workflow yes/no) in the table
- When I copied some of the workflows, as a user I can see them in the Dashboard --> Studies --> Templates in the Workflow List multiple times (needs a better filter mechanism)
- If I edit an existing workflow steps, the configuration json is not filled out, is it loaded correctly?
- Nice to have feature (low prio): When important from json and there are multiple workflows, select which one to import
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.
New User Features
A list of functionalities designed to enhance the user experience.
New Dev Features
A list of tools and capabilities aimed at improving developer productivity.
parentWorkflowIdforeign key for workflow versioning.hiddenInFrontendfield to control frontend visibility of workflows.Improvements
NA
Bug Fixes
NA
Known Limitations
Future Steps
*implement the import fucntionality (to be discussed)