semantic-kernel icon indicating copy to clipboard operation
semantic-kernel copied to clipboard

Add ability to read outlook emails to EmailSkill

Open sw47 opened this issue 2 years ago • 1 comments

Motivation and Context

  1. Why is this change required?

Today, there is no ability to get outlook email contents in SK.

  1. What problem does it solve?

Ability to get outlook email contents in SK.

  1. What scenario does it contribute to?

Ability to get specific emails from one's outlook inbox and perform actions (i.e. additional skills)

Description

Add the ability to read outlook email contents to SK Skill. Currently, this PR is in Draft state - please see the comments for questions on design/intention.

I will add full UT once those questions have been resolved.

Contribution Checklist

  • [x] The code builds clean without any errors or warnings
  • [x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
  • [x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
  • [ ] All unit tests pass, and I have added new tests where possible
  • [x] I didn't break anyone :smile:

sw47 avatar Apr 14 '23 19:04 sw47

I recommend using the int.TryParse() methods and consider the locale settings of the user using the CultureInfo class. The vast majority of people are not from the U.S.A.

MovGP0 avatar Apr 21 '23 14:04 MovGP0

FYI - we'll be bringing in this feature this week. I'll link to the PR when it get created before closing this PR.

adrianwyatt avatar May 02 '23 16:05 adrianwyatt

https://github.com/microsoft/semantic-kernel/pull/801

adrianwyatt avatar May 04 '23 04:05 adrianwyatt