Skip to content

docs: rewrite variable aggregator node page#745

Merged
RiskeyL merged 5 commits intomainfrom
docs/rewrite-variable-aggregator
Apr 15, 2026
Merged

docs: rewrite variable aggregator node page#745
RiskeyL merged 5 commits intomainfrom
docs/rewrite-variable-aggregator

Conversation

@RiskeyL
Copy link
Copy Markdown
Contributor

@RiskeyL RiskeyL commented Apr 15, 2026

Summary

  • Rewrite the Variable Aggregator page to clarify the node's purpose upfront and correct the misleading framing that it "combines" variables from multiple branches.
  • Add a callout explicitly stating the node is not for merging parallel branches, pointing users to Code/Template nodes instead.
  • Replace vague section headings ("Configuration", "Advanced Features") with task-oriented ones ("Select the Variables to Converge", "Converge Multiple Sets of Variables").
  • Add a paragraph length rule (under 50 words) to the style guide to prevent visually dense blocks on Mintlify.
  • Update zh and ja translations to match the new English source.

Copilot AI review requested due to automatic review settings April 15, 2026 01:04
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 15, 2026
@dosubot dosubot bot added the documentation Improvements or additions to documentation label Apr 15, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Rewrites the Variable Aggregator node documentation to clarify that it’s meant to converge exclusive branches (not merge parallel execution), updates supporting visuals and headings, and aligns JA/ZH translations with the new structure. Also adds a paragraph-length guideline to the writing style guide to improve readability on Mintlify.

Changes:

  • Reframe Variable Aggregator behavior around exclusive branching, with an explicit “not for parallel merge” callout and links to Code/Template nodes.
  • Replace prior example-heavy content with a shorter explanation and local screenshot assets.
  • Add a “keep paragraphs under 50 words” rule to the documentation style guide and update JA/ZH pages accordingly.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 7 comments.

File Description
en/use-dify/nodes/variable-aggregator.mdx Rewrites the English node page with updated framing, callout, headings, and local images.
zh/use-dify/nodes/variable-aggregator.mdx Updates Chinese page to match the new English structure and messaging.
ja/use-dify/nodes/variable-aggregator.mdx Updates Japanese page to match the new English structure; also renames the node title.
writing-guides/style-guide.md Adds a paragraph length rule (under 50 words) for Mintlify readability.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread en/use-dify/nodes/variable-aggregator.mdx
Comment thread en/use-dify/nodes/variable-aggregator.mdx Outdated
Comment thread en/use-dify/nodes/variable-aggregator.mdx Outdated
Comment thread en/use-dify/nodes/variable-aggregator.mdx
Comment thread zh/use-dify/nodes/variable-aggregator.mdx
Comment thread ja/use-dify/nodes/variable-aggregator.mdx
Comment thread ja/use-dify/nodes/variable-aggregator.mdx
RiskeyL and others added 5 commits April 15, 2026 11:07
Remove trailing whitespace in en page and restore translation
disclaimers on zh and ja pages for consistency with other translated
node pages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add a Translation Disclaimer section to both zh and ja formatting
guides. Every translated page must include the disclaimer regardless
of review status, since the English source is always canonical and
readers benefit from a consistent pointer on every page.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Simplify the guidance on using a Variable Aggregator to converge
plugin and webhook trigger branches. Align zh and ja translations
with the glossary term (变量聚合器 / 変数集約器).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@RiskeyL RiskeyL force-pushed the docs/rewrite-variable-aggregator branch from 4a9604b to 81ab0aa Compare April 15, 2026 03:25
@RiskeyL RiskeyL merged commit b94ec21 into main Apr 15, 2026
2 of 3 checks passed
@RiskeyL RiskeyL deleted the docs/rewrite-variable-aggregator branch April 15, 2026 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants