Skip to content

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

Merged
merged 19 commits into from
Jul 23, 2025

Conversation

wictorwilen
Copy link
Contributor

Added new on how to build a Copilot Connector for people data (preview)

@jasonjoh jasonjoh self-assigned this Jun 11, 2025
Copy link
Collaborator

@jasonjoh jasonjoh left a 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?

@wictorwilen
Copy link
Contributor Author

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).

@wictorwilen
Copy link
Contributor Author

@jasonjoh - where would a good location for the sample(s) to be added?

@wictorwilen
Copy link
Contributor Author

@jasonjoh - gentle nudge on this. We'd like to get this published this week.

@jasonjoh
Copy link
Collaborator

jasonjoh commented Jul 2, 2025

@jasonjoh - where would a good location for the sample(s) to be added?

I'd recommend getting repo created in GitHub, possibly in the microsoftgraph org. You should be able to request one in the open source portal.

@jasonjoh
Copy link
Collaborator

jasonjoh commented Jul 2, 2025

@jasonjoh - gentle nudge on this. We'd like to get this published this week.

I'm going to work on revising this as I suggested earlier and will ask for your review.

@wictorwilen
Copy link
Contributor Author

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.

@wictorwilen
Copy link
Contributor Author

Updated the sample code and referenced this article correctly. Awaiting the final privacy approvals before we can go live

@wictorwilen
Copy link
Contributor Author

Feature is rolled out in targeted release, and to WW right now. Publishing.

@wictorwilen wictorwilen reopened this Jul 22, 2025
@wictorwilen
Copy link
Contributor Author

@jasonjoh - please merge this.

@jasonjoh jasonjoh merged commit 4a073f9 into MicrosoftDocs:main Jul 23, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants