[AC-1423] Switch to AddonProductType enum instead of boolean

apps/web/src/app/billing/organizations/organization-subscription-cloud.component.html

@@ -70,10 +70,8 @@
         <ng-template body>
           <ng-container *ngIf="subscription">
             <tr bitRow *ngFor="let i of lineItems">
-              <td bitCell [ngClass]="{ 'tw-pl-20': i.addonSubscriptionItem }">
-                <span *ngIf="!i.addonSubscriptionItem"
-                  >{{ productName(i.bitwardenProduct) }} -</span
-                >
+              <td bitCell [ngClass]="{ 'tw-pl-20': i.addonProduct != null }">
+                <span *ngIf="i.addonProduct == null">{{ productName(i.bitwardenProduct) }} -</span>
                 {{ i.name }} {{ i.quantity > 1 ? "&times;" + i.quantity : "" }} @
                 {{ i.amount | currency : "$" }}
               </td>

apps/web/src/app/billing/organizations/organization-subscription-cloud.component.ts

@@ -349,22 +349,25 @@ export class OrganizationSubscriptionCloudComponent implements OnInit, OnDestroy
 }
 
 /**
- * Helper to sort subscription items by product type and then by addon status
+ * Helper to sort subscription items by product type and then by addon product type
  */
 function sortSubscriptionItems(
   a: BillingSubscriptionItemResponse,
   b: BillingSubscriptionItemResponse
 ) {
   if (a.bitwardenProduct == b.bitwardenProduct) {
-    // sort addon items to the bottom
-    if (a.addonSubscriptionItem == b.addonSubscriptionItem) {
-      return 0;
+    // Both are addon products, sort by enum value
+    if (a.addonProduct != null && b.addonProduct != null) {
+      return a.addonProduct - b.addonProduct;
     }
 
-    if (a.addonSubscriptionItem) {
-      return 1;
+    // 'a' is not an addon product, sort it to the bottom
+    if (a.addonProduct == null) {
+      return -1;
     }
-    return -1;
+
+    // 'a' is an addon product, sort it to the top
+    return 1;
   }
   return a.bitwardenProduct - b.bitwardenProduct;
 }

libs/common/src/billing/enums/addon-product-type.enum.ts

@@ -0,0 +1,8 @@
+/**
+ *  Used to identify the various types of "addon" products that can be added
+ *  to an existing product subscription.
+ */
+export enum AddonProductType {
+  PasswordManager_Storage = 0,
+  SecretsManager_ServiceAccounts = 1,
+}

libs/common/src/billing/models/response/subscription.response.ts

@@ -1,4 +1,5 @@
 import { BaseResponse } from "../../../models/response/base.response";
+import { AddonProductType } from "../../enums/addon-product-type.enum";
 import { BitwardenProductType } from "../../enums/bitwarden-product-type.enum";
 
 export class SubscriptionResponse extends BaseResponse {
@@ -63,7 +64,7 @@ export class BillingSubscriptionItemResponse extends BaseResponse {
   quantity: number;
   interval: string;
   sponsoredSubscriptionItem: boolean;
-  addonSubscriptionItem: boolean;
+  addonProduct?: AddonProductType;
   bitwardenProduct: BitwardenProductType;
 
   constructor(response: any) {
@@ -73,7 +74,7 @@ export class BillingSubscriptionItemResponse extends BaseResponse {
     this.quantity = this.getResponseProperty("Quantity");
     this.interval = this.getResponseProperty("Interval");
     this.sponsoredSubscriptionItem = this.getResponseProperty("SponsoredSubscriptionItem");
-    this.addonSubscriptionItem = this.getResponseProperty("AddonSubscriptionItem");
+    this.addonProduct = this.getResponseProperty("AddonProduct");
     this.bitwardenProduct = this.getResponseProperty("BitwardenProduct");
   }
 }