Skip to content

Commit d1e906e

Browse files
New-AzApiManagementProduct: Change SubscriptionsLimit parameter default value to None (#13457)
* Change SubscriptionsLimit default value to None * Update changelog * Fix * update changelog.md Co-authored-by: Yeming Liu <[email protected]>
1 parent 7d16f93 commit d1e906e

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/ApiManagement/ApiManagement.ServiceManagement/Commands/NewAzureApiManagementProduct.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ public class NewAzureApiManagementProduct : AzureApiManagementCmdletBase
7474
ValueFromPipelineByPropertyName = true,
7575
Mandatory = false,
7676
HelpMessage = "Maximum number of simultaneous subscriptions. This parameter is optional." +
77-
" Default value is 1.")]
77+
" Default value is None.")]
7878
public Int32? SubscriptionsLimit { get; set; }
7979

8080
[Parameter(
@@ -89,11 +89,9 @@ public override void ExecuteApiManagementCmdlet()
8989
string productId = ProductId ?? Guid.NewGuid().ToString("N");
9090

9191
bool? approvalRequired = null;
92-
Int32? subscriptionsLimit = null;
9392
if (SubscriptionRequired.HasValue && SubscriptionRequired.Value)
9493
{
9594
approvalRequired = ApprovalRequired ?? false;
96-
subscriptionsLimit = SubscriptionsLimit ?? 1;
9795
}
9896

9997
var product = Client.ProductCreate(
@@ -104,7 +102,7 @@ public override void ExecuteApiManagementCmdlet()
104102
LegalTerms,
105103
SubscriptionRequired ?? true,
106104
approvalRequired,
107-
subscriptionsLimit,
105+
SubscriptionsLimit,
108106
State);
109107

110108
WriteObject(product);

src/ApiManagement/ApiManagement/ChangeLog.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
- Additional information about change #1
1919
-->
2020
## Upcoming Release
21+
* [Breaking change] `New-AzApiManagementProduct` by default has no subscription limit.
2122

2223
## Version 2.1.0
2324
* Added new `Add-AzApiManagementApiToGateway` cmdlet.

0 commit comments

Comments
 (0)