Skip to content

fix: audio data folder name consistency#677

Merged
Avijit-Microsoft merged 3 commits into
devfrom
rc-audiodatabug
Dec 4, 2025
Merged

fix: audio data folder name consistency#677
Avijit-Microsoft merged 3 commits into
devfrom
rc-audiodatabug

Conversation

@Ragini-Microsoft

@Ragini-Microsoft Ragini-Microsoft commented Dec 4, 2025

Copy link
Copy Markdown
Contributor

Purpose

This pull request updates the deployment scripts and related Bicep module to simplify argument passing and fix a folder naming issue. The main changes involve removing an unused fileSystem argument, updating the order of script arguments, and correcting a folder name in the extraction process.

Deployment script argument changes:

  • The uploadFiles Bicep module in infra/main.bicep now omits the data argument and reorders the remaining arguments passed to the deployment script, reflecting the removal of the unused parameter.

  • Both infra/scripts/copy_kb_files.sh and infra/scripts/copy_kb_files_manual.sh have been updated to remove the fileSystem variable, and the order of remaining arguments has been adjusted to match the new invocation from Bicep. [1] [2]

Bug fix:

  • In infra/scripts/copy_kb_files.sh, the folder name for extracting audio data has been corrected from audiodata to audio_data to match the zip file contents.

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

@Avijit-Microsoft
Avijit-Microsoft merged commit f1ca837 into dev Dec 4, 2025
5 checks passed
@github-actions

github-actions Bot commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.16.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants