-
Notifications
You must be signed in to change notification settings - Fork 37
New article on building Copilot Connectors for people data (preview) #49
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
base: main
Are you sure you want to change the base?
Conversation
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.
Thanks for this contribution! By making this a walkthrough, we're adding a lot of overhead that isn't specific to the "people connector" concept like app registration and auth, connector creation, etc. At the end of the day, this is basically the same as any other connector, with just a specific requirement on the schema, and an extra Graph call to register it as a profile source. How would you feel about shortening this article to discuss those specific bits only, and then link to your completed sample in GitHub?
I don't mind cutting out some of code for readability and link to a full example (that's less to maintain). |
@jasonjoh - where would a good location for the sample(s) to be added? |
@jasonjoh - gentle nudge on this. We'd like to get this published this week. |
I'd recommend getting repo created in GitHub, possibly in the |
I'm going to work on revising this as I suggested earlier and will ask for your review. |
Thanks @jasonjoh for the feedback. I've updated the article, as per your request, to refer to a full sample code repo found at https://github.com/microsoftgraph/msgraph-people-connector-sample-dotnet. |
Updated the sample code and referenced this article correctly. Awaiting the final privacy approvals before we can go live |
Added new on how to build a Copilot Connector for people data (preview)