Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Copilot] Use managed deployment instead of deployment when setting first party authorization #1282

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

encimita
Copy link
Contributor

@encimita encimita commented Jun 10, 2024

Summary

The wrong deployment is set here, causing a bug in main branch for first party immersive Copilot features.

Work Item(s)

Fixes AB#537361

@encimita encimita requested a review from a team as a code owner June 10, 2024 11:36
@github-actions github-actions bot added this to the Version 25.0 milestone Jun 10, 2024
@encimita encimita requested a review from darjoo June 10, 2024 11:37
@encimita encimita merged commit 6daca64 into main Jun 10, 2024
29 checks passed
@encimita encimita deleted the private/encimita/ManagedDeploymentBug branch June 10, 2024 14:26
encimita added a commit that referenced this pull request Jun 11, 2024
…irst party authorization (#1282)

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
The wrong deployment is set here, causing a bug in main branch for first
party immersive Copilot features.

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#537361](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/537361)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants