Skip to content

Fix missing user_id in trace_manager#24024

Merged
crazywoola merged 1 commit intolanggenius:mainfrom
hyongtao-code:fix-trace
Aug 16, 2025
Merged

Fix missing user_id in trace_manager#24024
crazywoola merged 1 commit intolanggenius:mainfrom
hyongtao-code:fix-trace

Conversation

@hyongtao-code
Copy link
Copy Markdown
Contributor

Fixes #24023

Unlike AgentChatAppGenerator and AdvancedChatAppGenerator, the TraceQueueManager in ChatAppGenerator and CompletionAppGenerator is missing the user_id.

Signed-off-by: Yongtao Huang yongtaoh2022@gmail.com

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

Screenshots

Before After
... ...

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 dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 15, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 16, 2025
@crazywoola crazywoola merged commit 8b601a9 into langgenius:main Aug 16, 2025
7 checks passed
bowenliang123 pushed a commit to bowenliang123/dify that referenced this pull request Aug 18, 2025
@hyongtao-code hyongtao-code deleted the fix-trace branch August 27, 2025 05:35
qiqizjl pushed a commit to qiqizjl/dify that referenced this pull request Aug 27, 2025
HarryReidx pushed a commit to HarryReidx/dify that referenced this pull request Sep 1, 2025
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:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Chore] missing user_id in trace_manager

2 participants