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

Chore/Enhancement: Improve 'Require Approval' Description to Indicate MemoryAgent Dependency #3680

Merged

Conversation

jeanibarz
Copy link
Contributor

@jeanibarz jeanibarz commented Dec 10, 2024

This pull request updates the description of the 'Require Approval' trigger button in the Sequential Agent node settings to clarify its dependency on the MemoryAgent configuration. The updated description informs users that enabling the "Require Approval" option requires a defined MemoryAgent, helping prevent confusion and unnecessary troubleshooting.

Changes:

  • Updated the description of the 'Require Approval' option from:

    "Require approval before executing tools. Will proceed when tools are not called."

    To:

    "Pause execution and request user approval before running tools.
    If enabled, the agent will prompt the user with customizable approve/reject options
    and will proceed only after approval. This requires a configured MemoryAgent to manage
    the state and handle approval requests.
    If no tools are invoked, the agent proceeds without interruption."

Reason for Change:
The original description did not provide sufficient information, leading to non-intuitive error messages when MemoryAgent was not configured. This update makes the requirement explicit, improving user experience and reducing potential errors.

@jeanibarz jeanibarz force-pushed the chore/require-approval-description branch from 09e46be to 27ed7cf Compare December 10, 2024 21:48
@jeanibarz jeanibarz changed the title [Chore Enhancement] Improve 'Require Approval' Description to Indicate MemoryAgent DependencyChore/require approval description Chore/Enhancement: Improve 'Require Approval' Description to Indicate MemoryAgent Dependency Dec 10, 2024
Copy link
Contributor

@HenryHengZJ HenryHengZJ left a comment

Choose a reason for hiding this comment

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

thank you @jeanibarz !

@HenryHengZJ HenryHengZJ merged commit d549885 into FlowiseAI:main Dec 17, 2024
2 checks passed
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.

2 participants