Skip to content

Conversation

@Ravikirana-Microsoft
Copy link
Contributor

Purpose

This pull request makes a minor adjustment to the AppWrapper component in src/index.tsx by moving the sendUserBrowserLanguage API call to a different location within the try block to improve execution order and error handling.

Key change:

  • The sendUserBrowserLanguage API call was relocated from before the try block to within the try block, after setting the default user information. This ensures the call is made only if earlier operations succeed, improving error handling and logical flow. ([[1]](https://github.com/microsoft/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/pull/353/files#diff-d2f8515a5da5fdbaafcfe7f5be649fa68b891afe926f96bcc71a2d247cba04dfL26), [[2]](https://github.com/microsoft/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/pull/353/files#diff-d2f8515a5da5fdbaafcfe7f5be649fa68b891afe926f96bcc71a2d247cba04dfL42-R41))

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit 91ea5df into dev Jul 30, 2025
7 checks passed
blessing-msft pushed a commit that referenced this pull request Aug 1, 2025
@github-actions
Copy link

🎉 This PR is included in version 2.2.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