windmill-labs / windmill-labs/windmill

bug: "dataflow" toggle-able for "Inner flow" but not for flow itself

Open
#4,012 0 comments 0 reactions 1 assignee View on GitHub

@rubenfiszel is already working on this.

Since Jul 2, 2024.

bug
Dominant language
Rust
Stars
18k
Forks
1.1k
Avg merge
1d 1h
Merged PRs (30d)
328

Description

Describe the bug

I successfully enabled the "dataflow" feature on an "inner flow".

However, as it rendered weirdly, I thought I wanted to look into the sub-flow itself in fullscreen with the "dataflow" feature, but I cannot enable it as the toggle button is not presented to me.

To reproduce

Whilst trying to reproduce I noticed that the toggle-button seemingly randomly appeared and disappeared for both flows. I was not able to get it reproduced 1:1.

Expected behavior

Either it is toggle-able on both or none of the views.

And it should render in exactly the same way: The "dataflow" of the "inner flow" does not seem to render the Flow input correctly!

Screenshots

As inner flow
image

When opened the inner flow itself in the flow viewer:
image

Browser information

Chrome Version 126.0.6478.127

Application version

EE v1.355.3

Additional Context

Might not be related, but my developer console is spammed with:

Error: <path> attribute d: Expected number, "…,333 389.5,378 Mundefined,undefi…".

scheduler.BMtNMwrL.js

(Scheduler is not used for any of those flows!)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.