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 Haystack tool step text #956

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix Haystack tool step text #956

wants to merge 1 commit into from

Conversation

mosheber
Copy link

@mosheber mosheber commented May 1, 2024

Fixing a bug where the tool_result text is not displayed in the UI.

  • replaced the tool_result update by calling stream_token
  • replaced the id of the tool step with the parent id of the step above it, instead of the id itself.

Fixing a bug where the tool_result text is not displayed in the UI.
* replaced the tool_result update by calling stream_token
* replaced the id of the tool step with the parent id of the step above it, instead of the id itself.
@mosheber
Copy link
Author

mosheber commented May 8, 2024

@willydouhard is there something I should add to this PR?

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