Skip to content

docs: Added content in Readme and deploymentguide for SFI issue #402

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

Merged
Roopan-Microsoft merged 4 commits into from
Apr 10, 2026
Merged

docs: Added content in Readme and deploymentguide for SFI issue #402

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
8948c30
Merge pull request #390 from microsoft/dev
Avijit-Microsoft Apr 6, 2026
121791c
Merge pull request #398 from microsoft/dev
Roopan-Microsoft Apr 8, 2026
72a30d0
Update README
Thanusree-Microsoft Apr 9, 2026
9d16b7e
Update Deployment Guide
Thanusree-Microsoft Apr 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,8 @@ Follow the quick deploy steps on the deployment guide to deploy this solution to

<br/>

> **Note**: Some tenants may have additional security restrictions that run periodically and could impact the application (e.g., blocking public network access). If you experience issues or the application stops working, check if these restrictions are the cause. In such cases, consider deploying the WAF-supported version to ensure compliance. To configure, [Click here](./docs/DeploymentGuide.md#31-choose-deployment-type-optional).
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

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

The note refers to a "WAF-supported version", but elsewhere in the docs (and earlier in this README) the terminology is "WAF-aligned" / "WAF-Aligned deployment option". Consider aligning the wording here to the established term to avoid confusion about which deployment path to choose.

Suggested change
> **Note**: Some tenants may have additional security restrictions that run periodically and could impact the application (e.g., blocking public network access). If you experience issues or the application stops working, check if these restrictions are the cause. In such cases, consider deploying the WAF-supported version to ensure compliance. To configure, [Click here](./docs/DeploymentGuide.md#31-choose-deployment-type-optional).
> **Note**: Some tenants may have additional security restrictions that run periodically and could impact the application (e.g., blocking public network access). If you experience issues or the application stops working, check if these restrictions are the cause. In such cases, consider deploying the WAF-aligned version to ensure compliance. To configure, [Click here](./docs/DeploymentGuide.md#31-choose-deployment-type-optional).

Copilot uses AI. Check for mistakes.

> ⚠️ **Important: Check Azure OpenAI Quota Availability**
<br/>To ensure sufficient quota is available in your subscription, please follow [quota check instructions guide](./docs/quota_check.md) before you deploy the solution.

Expand Down
2 changes: 2 additions & 0 deletions docs/DeploymentGuide.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ This guide walks you through deploying the Modernize Your Code Solution Accelera

🆘 **Need Help?** If you encounter any issues during deployment, check our [Troubleshooting Guide](./TroubleShootingSteps.md) for solutions to common problems.

> **Note**: Some tenants may have additional security restrictions that run periodically and could impact the application (e.g., blocking public network access). If you experience issues or the application stops working, check if these restrictions are the cause. In such cases, consider deploying the WAF-supported version to ensure compliance. To configure, [Click here](#31-choose-deployment-type-optional).
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

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

This note mentions deploying the "WAF-supported version", but the rest of the documentation describes this as the "Production" / "WAF-aligned" deployment option. Please use the same term here (and/or explicitly reference "Production" in section 3.1) to avoid ambiguity.

Suggested change
> **Note**: Some tenants may have additional security restrictions that run periodically and could impact the application (e.g., blocking public network access). If you experience issues or the application stops working, check if these restrictions are the cause. In such cases, consider deploying the WAF-supported version to ensure compliance. To configure, [Click here](#31-choose-deployment-type-optional).
> **Note**: Some tenants may have additional security restrictions that run periodically and could impact the application (e.g., blocking public network access). If you experience issues or the application stops working, check if these restrictions are the cause. In such cases, consider deploying the Production (WAF-aligned) deployment option to ensure compliance. To configure, [Click here](#31-choose-deployment-type-optional).

Copilot uses AI. Check for mistakes.

## Step 1: Prerequisites & Setup

### 1.1 Azure Account Requirements
Expand Down
Loading