Skip to content

fix: bug fix - #353

Merged
Roopan-Microsoft merged 1 commit into
devfrom
bug-fixes
Jul 30, 2025
Merged

fix: bug fix#353
Roopan-Microsoft merged 1 commit into
devfrom
bug-fixes

Conversation

@Ravikirana-Microsoft

Copy link
Copy Markdown
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
Copy Markdown

🎉 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