Skip to content

Commit 4528080

Browse files
committed
code review feedback
1 parent 6a30804 commit 4528080

File tree

3 files changed

+6
-19
lines changed

3 files changed

+6
-19
lines changed

examples/lib/setup.sh

Lines changed: 0 additions & 17 deletions
This file was deleted.

src/CLU/Microsoft.Azure.Commands.Websites/Cmdlets/DeploymentSlots/SetAzureWebAppSlot.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,9 @@ public class SetAzureWebAppSlotCmdlet : WebAppSlotBaseCmdlet
6464

6565
[Parameter(Position = 11, Mandatory = false, HelpMessage = "Web app connection strings")]
6666
[ValidateNotNullOrEmpty]
67-
public Hashtable ConnectionStrings { get; set; }
67+
// TODO: CLU
68+
//public Hashtable ConnectionStrings { get; set; }
69+
public Dictionary<string, ConnStringValueTypePair> ConnectionStrings { get; set; }
6870

6971
[Parameter(Position = 12, Mandatory = false, HelpMessage = "Web app handler mappings")]
7072
[ValidateNotNullOrEmpty]
@@ -117,7 +119,7 @@ protected override void ProcessRecord()
117119
}
118120

119121
// Update web app configuration
120-
WebsitesClient.UpdateWebAppConfiguration(ResourceGroupName, location, Name, Slot, siteConfig, AppSettings.ConvertToStringDictionary(), ConnectionStrings.ConvertToConnectionStringDictionary());
122+
WebsitesClient.UpdateWebAppConfiguration(ResourceGroupName, location, Name, Slot, siteConfig, AppSettings.ConvertToStringDictionary(), ConnectionStrings);
121123

122124
if (parameters.Contains("AppServicePlan"))
123125
{

src/CLU/Microsoft.Azure.Commands.Websites/Cmdlets/WebApps/SetAzureWebApp.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,8 @@ public class SetAzureWebAppCmdlet : WebAppBaseCmdlet
6868
[Parameter(ParameterSetName = ParameterSet1Name, Position = 10, Mandatory = false, HelpMessage = "Web app connection strings. Example: -ConnectionStrings @{ ConnectionString1 = @{ Type = \"MySql\"; Value = \"MySql Connection string\"}; ConnectionString2 = @{ Type = \"SQLAzure\"; Value = \"SqlAzure Connection string 2\"} }")]
6969
[ValidateNotNullOrEmpty]
7070
[ValidateConnectionStrings]
71+
// TODO: CLU
72+
//public Hashtable ConnectionStrings { get; set; }
7173
public Dictionary<string, ConnStringValueTypePair> ConnectionStrings { get; set; }
7274

7375
[Parameter(ParameterSetName = ParameterSet1Name, Position = 11, Mandatory = false, HelpMessage = "Web app handler mappings")]

0 commit comments

Comments
 (0)