Skip to content

Conversation

@jakeminard
Copy link
Collaborator

CG-14843: Content Sync Language Always Set to En

In graph-source-sdk, content is always routed to English when synced. This seems to be hard coded (see code here)

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a hardcoded language issue where content sync was always using "en" as the language. The fix allows users to specify the language parameter when syncing content.

  • Adds a language parameter to the SaveContentAsync method throughout the codebase
  • Removes the hardcoded "en" language assignment in the content creation logic
  • Updates all method signatures, implementations, and test calls to include the new language parameter

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
README.md Adds documentation example showing how to use the new language parameter
IGraphSourceRepository.cs Updates interface to include language parameter in SaveContentAsync method
GraphSourceRepository.cs Removes hardcoded "en" language and accepts language as parameter
GraphSourceClient.cs Updates client method to pass through the language parameter
GraphSourceRepositoryTests.cs Updates test calls to include the language parameter
Program.cs Updates sample code to use the new method signature with language parameter

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jakeminard jakeminard merged commit f751c98 into main Sep 12, 2025
1 check 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.

6 participants