Skip to content

fix: remove "locales" from CU config to support multi language#688

Closed
cyberflying wants to merge 1 commit intomicrosoft:mainfrom
cyberflying:fix-cu-lang
Closed

fix: remove "locales" from CU config to support multi language#688
cyberflying wants to merge 1 commit intomicrosoft:mainfrom
cyberflying:fix-cu-lang

Conversation

@cyberflying
Copy link
Contributor

Purpose

  • Update the Audio Analyzer configuration to support multiple languages.

Previously, the configuration file ckm-analyzer_config_audio.json specified:
locales: ["en-US"]
This limited support to English only.
To enable multi-language support, remove the locales setting from the configuration.
Verified that the updated configuration works for both English and Chinese.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

Other Information

@cyberflying cyberflying changed the title remove "locales" from CU config to support multi language fix: remove "locales" from CU config to support multi language Dec 13, 2025
@Avijit-Microsoft
Copy link
Contributor

Hi @cyberflying,
Thank you for raising the PR. We’ve reviewed the proposed changes for handling all locales. The current implementation is intentionally limited to en-US and has been thoroughly tested and validated for this locale. Since it is working as expected, we do not plan to merge these changes at this time.

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.

2 participants