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

EmailClient: Updating attachment support and output #125

Closed
wants to merge 5 commits into from
Closed

Conversation

trish11953
Copy link
Contributor

@trish11953 trish11953 commented Oct 10, 2023

Resolves #123


📚 Documentation preview 📚: https://griptape--125.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
@trish11953 trish11953 changed the title Updated EmailClient docs that support attachments EmailClient: Updating attachment support and output Oct 10, 2023
smtp_host=os.environ.get("SMTP_HOST"),
smtp_port=int(os.environ.get("SMTP_PORT", 465)),
smtp_password=os.environ.get("SMTP_PASSWORD"),
smtp_user=os.environ.get("FROM_EMAIL"),
smtp_use_ssl=bool(os.environ.get("SMTP_USE_SSL")),
)

agent = Agent(
tools=[email_tool, WebScraper()],
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 WebScraper initialization next to email_tool so that all Tools are created in one place.

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!

)

agent.run(
"Scrape www.griptape.ai and and send it as an attachment in an email to "
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 we should show the summarization capabilities of Griptape as opposed to just emailing an HTML dump. Let's rephrase to something like Generate a summary of griptape.ai and send it as ...

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! Used ToolOutputProcessor too to store it back in memory

)

agent.run(
"Scrape www.griptape.ai and and send it as an attachment in an email to "
Copy link
Member

Choose a reason for hiding this comment

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

and and 🙂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my bad

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 EmailClient documentation with example output
2 participants