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

CopilotChat: Enable MsGraph ToDos in CopilotChat #781

Merged
merged 7 commits into from May 3, 2023

Conversation

adrianwyatt
Copy link
Contributor

@adrianwyatt adrianwyatt commented May 3, 2023

Motivation and Context

Enable MsGraph skills in CopilotChat.

Description

  • Added TaskListSkill to CopilotChat when MsGraph authentication is available
  • Reduced the webapp's scopes to just User.Read and Tasks.ReadWrite
  • Added a "GetDefaultTasks" function with filter for not getting completed tasks.
  • Fixed importing Klarna from ChatGPT plugin URL.
  • Removed unused methods from the OneDrive connector
  • Removed unused semantic skills from CopilotChat

@github-actions github-actions bot added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel samples labels May 3, 2023
@adrianwyatt adrianwyatt added the PR: ready for review All feedback addressed, ready for reviews label May 3, 2023
@adrianwyatt adrianwyatt marked this pull request as ready for review May 3, 2023 01:54
glahaye
glahaye previously approved these changes May 3, 2023
@adrianwyatt adrianwyatt self-assigned this May 3, 2023
glahaye
glahaye previously approved these changes May 3, 2023
@adrianwyatt adrianwyatt merged commit 562ab5b into microsoft:main May 3, 2023
11 checks passed
@@ -17,14 +18,16 @@ public static async Task RunAsync()
private static async Task RunChatGptPluginAsync()
{
var kernel = new KernelBuilder().WithLogger(ConsoleLogger.Log).Build();
using HttpClient importHttpClient = new HttpClient();
importHttpClient.DefaultRequestHeaders.Add("User-Agent", "Microsoft.SemanticKernel");
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be consistent in the value we use for user agent. Looks like elsewhere in this PR we are using "Microsoft-Semantic-Kernel"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch! I'll fix it in today's PR.

@adrianwyatt adrianwyatt deleted the msgraph-planner branch May 10, 2023 17:41
codebrain pushed a commit to searchpioneer/semantic-kernel that referenced this pull request May 16, 2023
Enable MsGraph skills in CopilotChat.

- Added TaskListSkill to CopilotChat when MsGraph authentication is
available
- Reduced the webapp's scopes to just User.Read and Tasks.ReadWrite
- Added a "GetDefaultTasks" function with filter for not getting
completed tasks.
- Fixed importing Klarna from ChatGPT plugin URL.
- Removed unused methods from the OneDrive connector
- Removed unused semantic skills from CopilotChat
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
### Motivation and Context
Enable MsGraph skills in CopilotChat.

### Description
- Added TaskListSkill to CopilotChat when MsGraph authentication is
available
- Reduced the webapp's scopes to just User.Read and Tasks.ReadWrite
- Added a "GetDefaultTasks" function with filter for not getting
completed tasks.
- Fixed importing Klarna from ChatGPT plugin URL.
- Removed unused methods from the OneDrive connector
- Removed unused semantic skills from CopilotChat
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
### Motivation and Context
Enable MsGraph skills in CopilotChat.

### Description
- Added TaskListSkill to CopilotChat when MsGraph authentication is
available
- Reduced the webapp's scopes to just User.Read and Tasks.ReadWrite
- Added a "GetDefaultTasks" function with filter for not getting
completed tasks.
- Fixed importing Klarna from ChatGPT plugin URL.
- Removed unused methods from the OneDrive connector
- Removed unused semantic skills from CopilotChat
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code PR: ready for review All feedback addressed, ready for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants