-
Notifications
You must be signed in to change notification settings - Fork 18
docs: Update Deployment Guide with Azure authentication steps #90
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
base: dev
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -120,6 +120,8 @@ Select one of the following options to deploy the Container Migration Solution A | |||||
| <details> | ||||||
| <summary><b>Option C: Visual Studio Code Web</b></summary> | ||||||
|
|
||||||
| ### VS Code Web | ||||||
|
|
||||||
| [&message=Open&color=blue&logo=visualstudiocode&logoColor=white)](https://insiders.vscode.dev/azure/?vscode-azure-exp=foundry&agentPayload=eyJiYXNlVXJsIjogImh0dHBzOi8vcmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbS9taWNyb3NvZnQvQ29udGFpbmVyLU1pZ3JhdGlvbi1Tb2x1dGlvbi1BY2NlbGVyYXRvci9yZWZzL2hlYWRzL21haW4vaW5mcmEvdnNjb2RlX3dlYiIsICJpbmRleFVybCI6ICIvaW5kZXguanNvbiIsICJ2YXJpYWJsZXMiOiB7ImFnZW50SWQiOiAiIiwgImNvbm5lY3Rpb25TdHJpbmciOiAiIiwgInRocmVhZElkIjogIiIsICJ1c2VyTWVzc2FnZSI6ICIiLCAicGxheWdyb3VuZE5hbWUiOiAiIiwgImxvY2F0aW9uIjogIiIsICJzdWJzY3JpcHRpb25JZCI6ICIiLCAicmVzb3VyY2VJZCI6ICIiLCAicHJvamVjdFJlc291cmNlSWQiOiAiIiwgImVuZHBvaW50IjogIiJ9LCAiY29kZVJvdXRlIjogWyJhaS1wcm9qZWN0cy1zZGsiLCAicHl0aG9uIiwgImRlZmF1bHQtYXp1cmUtYXV0aCIsICJlbmRwb2ludCJdfQ==) | ||||||
|
|
||||||
| 1. Click the badge above (may take a few minutes to load) | ||||||
|
|
@@ -132,13 +134,19 @@ Select one of the following options to deploy the Container Migration Solution A | |||||
| sh install.sh | ||||||
| ``` | ||||||
| During this process, you’ll be prompted with the message: | ||||||
| <br> Choose “**Overwrite with versions from template**” and provide a unique environment name when prompted. | ||||||
|
||||||
| <br> Choose “**Overwrite with versions from template**” and provide a unique environment name when prompted. | |
| <br> Choose “**Overwrite with versions from template**” and provide a unique environment name when prompted. |
Copilot
AI
Jan 8, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The step numbering is inconsistent. The sequence goes from step 5 (line 131) to step 7 (this line), skipping step 6. This should be renumbered as step 6 to maintain sequential ordering.
| 7. **Authenticate with Azure** (VS Code Web requires device code authentication): | |
| 6. **Authenticate with Azure** (VS Code Web requires device code authentication): |
Copilot
AI
Jan 8, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The step numbering is inconsistent. This should be step 7 instead of step 8, following the renumbering of the previous authentication step.
| 8. Proceed to [Step 3: Configure Deployment Settings](#step-3-configure-deployment-settings) | |
| 7. Proceed to [Step 3: Configure Deployment Settings](#step-3-configure-deployment-settings) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This heading appears redundant since the parent collapsible section is already titled "Option C: Visual Studio Code Web". Consider removing this heading or renaming it to something more specific like "Steps" to avoid duplication.