Fix name validation for containerized function apps (#4012)

* Fix name validation * Change casing * use ternary * upgrade appservice package * push broke * try again
microsoft · Mar 20, 2024 · 96b2972 · 96b2972
1 parent f91a838
commit 96b2972
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1178,7 +1178,7 @@
         "@azure/core-client": "^1.7.3",
         "@azure/core-rest-pipeline": "^1.11.0",
         "@azure/storage-blob": "^12.5.0",
-        "@microsoft/vscode-azext-azureappservice": "^3.1.1",
+        "@microsoft/vscode-azext-azureappservice": "^3.1.2",
         "@microsoft/vscode-azext-azureappsettings": "^0.2.1",
         "@microsoft/vscode-azext-azureutils": "^3.0.0",
         "@microsoft/vscode-azext-serviceconnector": "^0.1.3",

diff --git a/src/tree/SubscriptionTreeItem.ts b/src/tree/SubscriptionTreeItem.ts
@@ -108,8 +108,6 @@ export class SubscriptionTreeItem extends SubscriptionTreeItemBase {
         const promptSteps: AzureWizardPromptStep<IAppServiceWizardContext>[] = [];
         const executeSteps: AzureWizardExecuteStep<IAppServiceWizardContext>[] = [];
 
-        promptSteps.push(new SiteNameStep("functionApp"));
-
         const storageAccountCreateOptions: INewStorageAccountDefaults = {
             kind: StorageAccountKind.Storage,
             performance: StorageAccountPerformance.Standard,
@@ -118,6 +116,8 @@ export class SubscriptionTreeItem extends SubscriptionTreeItemBase {
 
         await detectDockerfile(context);
 
+        promptSteps.push(new SiteNameStep(context.dockerfilePath ? "containerizedFunctionApp" : "functionApp"));
+
         if (context.dockerfilePath) {
             const containerizedfunctionAppWizard = await createContainerizedFunctionAppWizard();
             promptSteps.push(...containerizedfunctionAppWizard.promptSteps);