Skip to content

Commit f390063

Browse files
authoredJan 5, 2024
Merge pull request #629 from santhoshb-msft/sb-release750update
updated install/upgrade to release mode and remove unused SupportMeteredBilling
2 parents 1d3d1ef + ae207dc commit f390063

File tree

7 files changed

+12
-22
lines changed

7 files changed

+12
-22
lines changed
 

‎deployment/Deploy.ps1

+3-3
Original file line numberDiff line numberDiff line change
@@ -305,13 +305,13 @@ if (!($ADMTApplicationID)) {
305305
Write-host "📜 Prepare publish files for the application"
306306
if (!(Test-Path '../Publish')) {
307307
Write-host " 🔵 Preparing Admin Site"
308-
dotnet publish ../src/AdminSite/AdminSite.csproj -c debug -o ../Publish/AdminSite/ -v q
308+
dotnet publish ../src/AdminSite/AdminSite.csproj -c release -o ../Publish/AdminSite/ -v q
309309

310310
Write-host " 🔵 Preparing Metered Scheduler"
311-
dotnet publish ../src/MeteredTriggerJob/MeteredTriggerJob.csproj -c debug -o ../Publish/AdminSite/app_data/jobs/triggered/MeteredTriggerJob/ -v q --runtime win-x64 --self-contained true
311+
dotnet publish ../src/MeteredTriggerJob/MeteredTriggerJob.csproj -c release -o ../Publish/AdminSite/app_data/jobs/triggered/MeteredTriggerJob/ -v q --runtime win-x64 --self-contained true
312312

313313
Write-host " 🔵 Preparing Customer Site"
314-
dotnet publish ../src/CustomerSite/CustomerSite.csproj -c debug -o ../Publish/CustomerSite/ -v q
314+
dotnet publish ../src/CustomerSite/CustomerSite.csproj -c release -o ../Publish/CustomerSite/ -v q
315315

316316
Write-host " 🔵 Zipping packages"
317317
Compress-Archive -Path ../Publish/AdminSite/* -DestinationPath ../Publish/AdminSite.zip -Force

‎deployment/Upgrade.ps1

+5-3
Original file line numberDiff line numberDiff line change
@@ -92,11 +92,11 @@ Write-host "#### Database Deployment complete ####"
9292

9393
Write-host "#### Deploying new code ####"
9494

95-
dotnet publish ../src/AdminSite/AdminSite.csproj -v q -c debug -o ../Publish/AdminSite/
95+
dotnet publish ../src/AdminSite/AdminSite.csproj -v q -c release -o ../Publish/AdminSite/
9696
Write-host "## Admin Portal built"
97-
dotnet publish ../src/MeteredTriggerJob/MeteredTriggerJob.csproj -v q -c debug -o ../Publish/AdminSite/app_data/jobs/triggered/MeteredTriggerJob --runtime win-x64 --self-contained true
97+
dotnet publish ../src/MeteredTriggerJob/MeteredTriggerJob.csproj -v q -c release -o ../Publish/AdminSite/app_data/jobs/triggered/MeteredTriggerJob --runtime win-x64 --self-contained true
9898
Write-host "## Metered Scheduler to Admin Portal Built"
99-
dotnet publish ../src/CustomerSite/CustomerSite.csproj -v q -c debug -o ../Publish/CustomerSite
99+
dotnet publish ../src/CustomerSite/CustomerSite.csproj -v q -c release -o ../Publish/CustomerSite
100100
Write-host "## Customer Portal Built"
101101

102102
Compress-Archive -Path ../Publish/CustomerSite/* -DestinationPath ../Publish/CustomerSite.zip -Force
@@ -121,3 +121,5 @@ Write-host "## Deployed code to Customer Portal"
121121

122122
Remove-Item -Path ../Publish -recurse -Force
123123
Write-host "#### Code deployment complete ####"
124+
125+
Write-host "#### Warning: If the upgrade is to >=7.5.0, MeterScheduler feature is pre-enabled through DB config instead of the App Service. Please update the App config accordingly. ####"

‎src/AdminSite/Startup.cs

+1-2
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,7 @@ public void ConfigureServices(IServiceCollection services)
7777
Resource = this.Configuration["SaaSApiConfiguration:Resource"],
7878
SaaSAppUrl = this.Configuration["SaaSApiConfiguration:SaaSAppUrl"],
7979
SignedOutRedirectUri = this.Configuration["SaaSApiConfiguration:SignedOutRedirectUri"],
80-
TenantId = this.Configuration["SaaSApiConfiguration:TenantId"] ?? Guid.Empty.ToString(),
81-
SupportMeteredBilling = Convert.ToBoolean(this.Configuration["SaaSApiConfiguration:supportmeteredbilling"])
80+
TenantId = this.Configuration["SaaSApiConfiguration:TenantId"] ?? Guid.Empty.ToString()
8281
};
8382
var knownUsers = new KnownUsersModel()
8483
{

‎src/AdminSite/appsettings.json

+1-2
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@
1919
// "signedoutredirecturi": "",
2020
// "fulfillmentapiversion": "2018-08-31",
2121
// "adauthenticationendpoint": "https://login.microsoftonline.com",
22-
// "saasappurl": "",
23-
// "supportmeteredbilling": true
22+
// "saasappurl": ""
2423
//},
2524
//"connectionstrings": {
2625
// "defaultconnection": ""

‎src/MeteredTriggerJob/Program.cs

+1-2
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,7 @@ static void Main (string[] args)
3636
ClientSecret = configuration["SaaSApiConfiguration:ClientSecret"],
3737
GrantType = configuration["SaaSApiConfiguration:GrantType"],
3838
Resource = configuration["SaaSApiConfiguration:Resource"],
39-
TenantId = configuration["SaaSApiConfiguration:TenantId"],
40-
SupportMeteredBilling = Convert.ToBoolean(configuration["SaaSApiConfiguration:SupportMeteredBilling"])
39+
TenantId = configuration["SaaSApiConfiguration:TenantId"]
4140
};
4241

4342
var creds = new ClientSecretCredential(config.TenantId.ToString(), config.ClientId.ToString(), config.ClientSecret);

‎src/MeteredTriggerJob/appsettings.json

+1-2
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,7 @@
66
"clientid": "",
77
"clientsecret": "",
88
"resource": "",
9-
"adauthenticationendpoint": "https://login.microsoftonline.com",
10-
"supportmeteredbilling": true
9+
"adauthenticationendpoint": "https://login.microsoftonline.com"
1110
},
1211
"connectionstrings": {
1312
"defaultconnection": ""

‎src/Services/Configurations/SaaSApiClientConfiguration.cs

-8
Original file line numberDiff line numberDiff line change
@@ -96,14 +96,6 @@ public class SaaSApiClientConfiguration
9696
/// </value>
9797
public string SaaSAppUrl { get; set; }
9898

99-
/// <summary>
100-
/// Gets or sets the saas metered billing support.
101-
/// </summary>
102-
/// <value>
103-
/// True or false.
104-
/// </value>
105-
public bool SupportMeteredBilling { get; set; }
106-
10799
/// <summary>
108100
/// Initializes or Gets the current run environment. Set to "development" or "production" is assumed.
109101
/// </summary>

0 commit comments

Comments
 (0)
Please sign in to comment.