Skip to content

fix: var reference picker can not choose sub vars#35732

Merged
iamjoel merged 3 commits into
mainfrom
fix/cannot-choose-sub-vars
Apr 30, 2026
Merged

fix: var reference picker can not choose sub vars#35732
iamjoel merged 3 commits into
mainfrom
fix/cannot-choose-sub-vars

Conversation

@iamjoel
Copy link
Copy Markdown
Contributor

@iamjoel iamjoel commented Apr 30, 2026

Fix the first issue of #35730

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint && make type-check (backend) and cd web && pnpm exec vp staged (frontend) to appease the lint gods

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 30, 2026
@github-actions github-actions Bot added the web This relates to changes on the web. label Apr 30, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.66%. Comparing base (11c52e9) to head (223388d).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #35732   +/-   ##
=======================================
  Coverage   85.66%   85.66%           
=======================================
  Files        4455     4455           
  Lines      208954   208957    +3     
  Branches    39036    39038    +2     
=======================================
+ Hits       178997   179005    +8     
+ Misses      26790    26785    -5     
  Partials     3167     3167           
Flag Coverage Δ
dify-ui 93.01% <ø> (ø)
web 86.45% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 30, 2026
@iamjoel iamjoel added this pull request to the merge queue Apr 30, 2026
Merged via the queue into main with commit 0ca3391 Apr 30, 2026
33 checks passed
@iamjoel iamjoel deleted the fix/cannot-choose-sub-vars branch April 30, 2026 08:39
iamjoel added a commit that referenced this pull request Apr 30, 2026
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants