Deployment plan view fixes - #1627
Conversation
| import { ext } from "../../../extensionVariables"; | ||
| import { CopilotOnRailsContext } from "../../../utils/copilotOnRails/CopilotOnRailsContext"; | ||
| import { DEPLOYMENT_PLAN_FILE_GLOB } from "../../../tree/project/projectPlanFiles"; | ||
| import { CopilotOnRailsContext } from "../../../utils/copilotOnRails/CopilotOnRailsContext"; |
There was a problem hiding this comment.
Any idea why this got moved? I don't see any obvious difference
There was a problem hiding this comment.
Probably just formatting differences I'll move it back
| { name: 'East US 2', code: 'eastus2' }, | ||
| { name: 'West US', code: 'westus' }, | ||
| { name: 'West US 2', code: 'westus2' }, | ||
| { name: 'West US 3', code: 'westus3' }, |
There was a problem hiding this comment.
I know that it's been like this for a while, but I feel like we should get this static list from the locations step or something similar in the tools package rather than managing it
|
Making more changes so waiting to get those into this PR before merging |
|
The above changes include some updates to the view to better align with what the azure-prepare skill flow looks like. I also removed the already approved plan functionality as it was super buggy since the status could be different with each iteration which made it so users could not continue. |
3 participants
Fixes: #1622
I also noticed the subscription and location had not been properly being shown on the view even if they have been chosen in the chat and show up in the plan.md. I tested this and it seemed to fix it.