[elsa] fix(LlmNodeState): add conditional enableLog control and initialization#156
Merged
CodeCasterX merged 1 commit intoJun 10, 2025
Conversation
XinLi-cyber
approved these changes
Jun 10, 2025
Member
|
该PR涉及app-platform仓库的Issue: #237 |
wjn1584
approved these changes
Jun 10, 2025
4 tasks
Yager-42
pushed a commit
to Yager-42/fit-framework
that referenced
this pull request
Aug 16, 2025
surpercodehang
pushed a commit
that referenced
this pull request
Oct 10, 2025
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.
🔗 相关问题 / Related Issue
Issue 链接 / Issue Link: #237
📋 变更类型 / Type of Change
📝 变更目的 / Purpose of the Change
升级到社区版本后,老应用升级后无法打开编排页面,同时问题回答会回答两遍。该变更修复上述问题,增强了对enableLog的校验,大模型节点也默认增加了enableLog这个结构体。
After upgrading to the Community Edition, legacy applications fail to load the orchestration page post-upgrade while experiencing duplicate responses in Q&A outputs. This change fixes the aforementioned issues by:
📋 主要变更 / Brief Changelog
Checkbox for enableLog now only renders when enableLogData exists
Prevents UI errors when logging functionality is unavailable
New addEnableLog() method ensures enableLog parameter exists
Defaults to true for new configurations
Seamlessly integrates with existing parameter handling
Checks for existing enableLog before initialization
Follows established patterns from other parameter ensure methods
Preserves all existing process() workflow functionality
🧪 验证变更 / Verifying this Change
测试步骤 / Test Steps
测试覆盖 / Test Coverage
📸 截图 / Screenshots
✅ 贡献者检查清单 / Contributor Checklist
请确保你的 Pull Request 符合以下要求 / Please ensure your Pull Request meets the following requirements:
基本要求 / Basic Requirements:
代码质量 / Code Quality:
测试要求 / Testing Requirements:
mvn -B clean package -Dmaven.test.skip=true,elsa README 中的编译检查 / Basic checks passmvn clean install/ Unit tests pass文档和兼容性 / Documentation and Compatibility:
📋 附加信息 / Additional Notes
审查者注意事项 / Reviewer Notes: