From f1e9f96e27f0c7e6bda7950f4d1600f91376824e Mon Sep 17 00:00:00 2001 From: Shane Melton Date: Thu, 22 Jun 2023 14:12:11 -0700 Subject: [PATCH] [AC-1418] Fix conflicting required/optional labels for auto-scaling limits --- .../secrets-manager/sm-adjust-subscription.component.ts | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/apps/web/src/app/billing/organizations/secrets-manager/sm-adjust-subscription.component.ts b/apps/web/src/app/billing/organizations/secrets-manager/sm-adjust-subscription.component.ts index fee31cc9d69..99f8fda1ad8 100644 --- a/apps/web/src/app/billing/organizations/secrets-manager/sm-adjust-subscription.component.ts +++ b/apps/web/src/app/billing/organizations/secrets-manager/sm-adjust-subscription.component.ts @@ -105,17 +105,14 @@ export class SecretsManagerAdjustSubscriptionComponent implements OnInit, OnDest const serviceAccountLimitControl = this.formGroup.controls.serviceAccountLimit; if (value.limitSeats) { - seatLimitControl.setValidators([Validators.required, Validators.min(value.seatCount)]); + seatLimitControl.setValidators([Validators.min(value.seatCount)]); seatLimitControl.enable({ emitEvent: false }); } else { seatLimitControl.disable({ emitEvent: false }); } if (value.limitServiceAccounts) { - serviceAccountLimitControl.setValidators([ - Validators.required, - Validators.min(value.serviceAccountCount), - ]); + serviceAccountLimitControl.setValidators([Validators.min(value.serviceAccountCount)]); serviceAccountLimitControl.enable({ emitEvent: false }); } else { serviceAccountLimitControl.disable({ emitEvent: false });