Skip to content

[26.2] Remove allowlisting for Toolkit AI Resources #3927

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 20, 2025
Merged

[26.2] Remove allowlisting for Toolkit AI Resources #3927

merged 1 commit into from
Jun 20, 2025
+15 −59

Conversation

encimita
Copy link
Contributor

@encimita encimita commented Jun 18, 2025 •
edited by github-actions bot
Loading

Summary

We are ready to open up for Public Preview.

Work Item(s)

Fixes AB#581214

@encimita
[26.x] Remove allowlisting for AI resources (2) (#3630)
caa48e9 
<!-- 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 -->
__BACKPORT:__
Remove the allowlisting logic from the toolkit. We target next minor
update with these changes.

This is a cherry-pick of the previously reverted changes.

#### 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#573846](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/573846)
@encimita encimita requested review from a team as code owners June 18, 2025 14:57
@encimita encimita changed the base branch from main to releases/26.2 June 18, 2025 14:57
@github-actions github-actions bot added Build: Automation Workflows and other setup in .github folder AL: System Application Build: scripts & configs Build scripts and configuration files AL: Business Foundation labels Jun 18, 2025
@github-actions github-actions bot modified the milestones: Version 27.0, Version 26.2 Jun 18, 2025
msft-sam
msft-sam approved these changes Jun 19, 2025
View reviewed changes
Copy link
Contributor

@msft-sam msft-sam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be fine for 26.2 but be aware you might conflict with #3925

@encimita encimita enabled auto-merge (squash) June 20, 2025 10:57
@encimita encimita removed the request for review from a team June 20, 2025 11:38
@encimita encimita requested review from a team and removed request for a team June 20, 2025 11:40
@encimita encimita closed this Jun 20, 2025
auto-merge was automatically disabled June 20, 2025 11:46

Pull request was closed

@encimita encimita reopened this Jun 20, 2025
@github-actions github-actions bot removed Build: Automation Workflows and other setup in .github folder Build: scripts & configs Build scripts and configuration files AL: Business Foundation labels Jun 20, 2025
@github-actions github-actions bot modified the milestones: Version 27.0, Version 26.2 Jun 20, 2025
@encimita encimita merged commit 5624ee9 into releases/26.2 Jun 20, 2025
29 checks passed
@encimita encimita deleted the private/encimita/remove-allowlisting branch June 20, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MonicaPAhuja MonicaPAhuja MonicaPAhuja approved these changes

@ankosiorek ankosiorek ankosiorek approved these changes

@msft-sam msft-sam msft-sam approved these changes

Assignees
No one assigned
Labels
AL: System Application
Projects
None yet
Milestone
Version 26.2
Development

Successfully merging this pull request may close these issues.
4 participants
@encimita @MonicaPAhuja @ankosiorek @msft-sam