-
Notifications
You must be signed in to change notification settings - Fork 18
GoogleGmailClient: Updating attachment support #126
Conversation
"Create a draft email in Gmail to [email protected] with the subject 'Test Draft', the body " | ||
"'This is a test draft email.' and also add an attachment from my local machine with path " | ||
"'/test/foo/bar.txt'.", | ||
"I want sample1.txt and sample2.txt to be sent as attachments to my email to [email protected] " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I prefer the original prompt phrasing, can we tweak that to include multiple attachments?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed it back with multiple attachments
@@ -26,42 +26,58 @@ gmail_tool = GoogleGmailClient( | |||
|
|||
# Set up an agent using the GoogleGmailClient tool | |||
agent = Agent( | |||
tools=[gmail_tool] | |||
tools=[FileManager(), gmail_tool] | |||
) | |||
|
|||
# Task: Create a draft email in GMail |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change GMail
to Gmail
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done!
@@ -26,42 +26,58 @@ gmail_tool = GoogleGmailClient( | |||
|
|||
# Set up an agent using the GoogleGmailClient tool | |||
agent = Agent( | |||
tools=[gmail_tool] | |||
tools=[FileManager(), gmail_tool] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move FileManager
initialization next to gmail_tool
so that the Tools are consistently initialized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok!
Resolves #124
📚 Documentation preview 📚: https://griptape--126.org.readthedocs.build/