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

fix(plugin-workflow): fix loop variable #2211

Merged
merged 1 commit into from
Jul 7, 2023
Merged

fix(plugin-workflow): fix loop variable #2211

merged 1 commit into from
Jul 7, 2023

Conversation

mytharcher
Copy link
Contributor

Description (Bug 描述)

Variable for loop item should be same as loop target item.

Steps to reproduce (复现步骤)

  1. Add loop node.
  2. Add any node which could use variable to the loop.
  3. Select scope variable in added node.

Expected behavior (预期行为)

"Loop item" should be same level as loop target.

Actual behavior (实际行为)

One more level as node id in the variable options.

Related issues (相关 issue)

#2157.

Reason (原因)

Node variable has been refactored, loop variables missed.

Solution (解决方案)

Also refactor loop node on scope variable options.

@height
Copy link

height bot commented Jul 7, 2023

This pull request has been linked to 1 task:

  • T-874 动态表达式数据源选择查询多条数据时,变量路径错误 (unlink task)

💡Tip: Add "Close T-874" to the pull request title or description, to a commit message, or in a comment to mark this task as "Done" when the pull request is merged.

@mytharcher mytharcher merged commit 6b220c3 into main Jul 7, 2023
4 checks passed
@mytharcher mytharcher deleted the T-874 branch July 7, 2023 14:52
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.

None yet

1 participant