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

[24.x][CopilotToolkit] Allow allowlisted tenants to get the first party deployment names #982

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

encimita
Copy link
Contributor

@encimita encimita commented Apr 24, 2024

Summary

Backporting: In previous workshops and in general for future usage, we want the AOAI deployment names to be accessible to tenants that are allowlisted.

Work Item(s)

Fixes AB#527326

…loyment names (#978)

<!-- 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
-->
In previous workshops and in general for future usage, we want the AOAI
deployment names to be accessible to tenants that are allowlisted.

needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#524618](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/524618)
@encimita encimita requested a review from a team as a code owner April 24, 2024 10:12
@encimita encimita changed the title [CopilotToolkit] Allow allowlisted tenants to get the first party dep… [24.x][CopilotToolkit] Allow allowlisted tenants to get the first party deployment names Apr 24, 2024
@github-actions github-actions bot added this to the Version 24.1 milestone Apr 24, 2024
@encimita encimita merged commit 6d3e5a5 into releases/24.x Apr 24, 2024
29 checks passed
@encimita encimita deleted the private/encimita/backport24-allowlist branch April 24, 2024 13:07
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