Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

GoogleGmailClient: Updating attachment support #126

Closed
wants to merge 3 commits into from
Closed

Conversation

trish11953
Copy link
Contributor

@trish11953 trish11953 commented Oct 10, 2023

Resolves #124


📚 Documentation preview 📚: https://griptape--126.org.readthedocs.build/

@trish11953 trish11953 self-assigned this Oct 10, 2023
@trish11953 trish11953 added the documentation Improvements or additions to documentation label Oct 10, 2023
"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] "
Copy link
Member

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?

Copy link
Contributor Author

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
Copy link
Member

Choose a reason for hiding this comment

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

Change GMail to Gmail

Copy link
Contributor Author

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]
Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update GoogleGmailClient documentation with attachment
2 participants