@autologix-engineering/shared 1.2.353 → 1.2.354

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autologix-engineering/shared",
-  "version": "1.2.353",
+  "version": "1.2.354",
   "dependencies": {
     "tslib": "^2.3.0"
   },

package/src/lib/data-transfer-objects/src/update.vendor.city.status.dto.d.ts

@@ -2,22 +2,36 @@ import { GenericStatus } from '../../enums/src/index';
 /** Longest reason text the dialog will accept. */
 export declare const VENDOR_CITY_STATUS_REASON_MAX_LENGTH = 255;
 /**
- * Marks one city Active / Inactive for a vendor (AD-3211).
+ * Marks one Operational Info row Active / Inactive for a vendor (AD-3211).
  *
- * The Operational Info tab can hold several rows for the same city (one per
- * business type), so this command deliberately keys on (vendor, city) and flips
- * every row for that pair — the user acts on a *city*, not on a business-type
- * row.
+ * Keyed on (vendor, city, businessType) — one row of the grid, which is what
+ * the user actually clicked. An earlier version keyed on (vendor, city) alone
+ * and flipped every business type for that city together; a vendor serving a
+ * city under five arrangements lost all five from one click, all stamped with
+ * the same reason. Roughly a quarter of configured vendor-cities carry more
+ * than one business type, so that was not a rare case.
+ *
+ * `businessType` is required rather than optional on purpose: an omitted value
+ * would otherwise widen the command back to every row for the city, which is
+ * the exact bug this replaced.
  *
  * `remark` carries the reason text, picked from the global
  * `ReasonMaster.VendorCityStatus` list or typed free-hand. It is the same shape
  * every other status-reason flow in the product uses (vendor, driver and city
  * profile status all store the chosen reason as text), so there is no reason id
- * to resolve and no second master list to keep in step.
+ * to resolve and no second master list to keep in step. It is recorded against
+ * the single row this command touches, so each business type keeps the reason
+ * it was actually withdrawn for.
  */
 export declare class UpdateVendorCityStatusDto {
     vendorPersonaId: string;
     cityId: string;
+    /**
+     * The Operational Info row's business type, e.g. `Rate Card (RC)`. Matches
+     * `VendorOperations.bizType`, which stores the plain label rather than an
+     * enum key, so this is a string and not `@IsEnum`.
+     */
+    businessType: string;
     status: GenericStatus;
     /** Required when marking Inactive; ignored when reactivating. */
     remark?: string;

package/src/lib/data-transfer-objects/src/update.vendor.city.status.dto.js

@@ -5,18 +5,26 @@ import { GenericStatus } from '../../enums/src/index';
 /** Longest reason text the dialog will accept. */
 export const VENDOR_CITY_STATUS_REASON_MAX_LENGTH = 255;
 /**
- * Marks one city Active / Inactive for a vendor (AD-3211).
+ * Marks one Operational Info row Active / Inactive for a vendor (AD-3211).
  *
- * The Operational Info tab can hold several rows for the same city (one per
- * business type), so this command deliberately keys on (vendor, city) and flips
- * every row for that pair — the user acts on a *city*, not on a business-type
- * row.
+ * Keyed on (vendor, city, businessType) — one row of the grid, which is what
+ * the user actually clicked. An earlier version keyed on (vendor, city) alone
+ * and flipped every business type for that city together; a vendor serving a
+ * city under five arrangements lost all five from one click, all stamped with
+ * the same reason. Roughly a quarter of configured vendor-cities carry more
+ * than one business type, so that was not a rare case.
+ *
+ * `businessType` is required rather than optional on purpose: an omitted value
+ * would otherwise widen the command back to every row for the city, which is
+ * the exact bug this replaced.
  *
  * `remark` carries the reason text, picked from the global
  * `ReasonMaster.VendorCityStatus` list or typed free-hand. It is the same shape
  * every other status-reason flow in the product uses (vendor, driver and city
  * profile status all store the chosen reason as text), so there is no reason id
- * to resolve and no second master list to keep in step.
+ * to resolve and no second master list to keep in step. It is recorded against
+ * the single row this command touches, so each business type keeps the reason
+ * it was actually withdrawn for.
  */
 export class UpdateVendorCityStatusDto {
 }
@@ -28,6 +36,11 @@ __decorate([
     IsUUID(),
     __metadata("design:type", String)
 ], UpdateVendorCityStatusDto.prototype, "cityId", void 0);
+__decorate([
+    IsString(),
+    IsNotEmpty({ message: 'A business type is required' }),
+    __metadata("design:type", String)
+], UpdateVendorCityStatusDto.prototype, "businessType", void 0);
 __decorate([
     IsEnum(GenericStatus),
     __metadata("design:type", String)

package/src/lib/data-transfer-objects/src/update.vendor.city.status.dto.js.map

@@ -1 +1 @@
-{"version":3,"file":"update.vendor.city.status.dto.js","sourceRoot":"","sources":["../../../../../../../libs/shared/src/lib/data-transfer-objects/src/update.vendor.city.status.dto.ts"],"names":[],"mappings":";AAAA,OAAO,EACN,MAAM,EACN,UAAU,EACV,UAAU,EACV,QAAQ,EACR,MAAM,EACN,SAAS,EACT,UAAU,GACV,MAAM,iBAAiB,CAAC;AACzB,OAAO,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAE9C,OAAO,EAAE,aAAa,EAAE,MAAM,uBAAuB,CAAC;AAEtD,kDAAkD;AAClD,MAAM,CAAC,MAAM,oCAAoC,GAAG,GAAG,CAAC;AAExD;;;;;;;;;;;;;GAaG;AACH,MAAM,OAAO,yBAAyB;CA4BrC;AA1BA;IADC,MAAM,EAAE;;kEACe;AAGxB;IADC,MAAM,EAAE;;yDACM;AAGf;IADC,MAAM,CAAC,aAAa,CAAC;;yDACA;AAetB;IAZC,UAAU,CACV,CAAC,GAA8B,EAAE,EAAE,CAClC,GAAG,CAAC,MAAM,KAAK,aAAa,CAAC,QAAQ,CACtC;IACA,QAAQ,EAAE;IACV,UAAU,CAAC,EAAE,OAAO,EAAE,8CAA8C,EAAE,CAAC;IACvE,SAAS,CAAC,oCAAoC,CAAC;IAChD,2EAA2E;IAC3E,2EAA2E;IAC3E,sEAAsE;IACtE,kEAAkE;;IACjE,SAAS,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;;yDACzD;AAIhB;IAFC,UAAU,EAAE;IACZ,MAAM,EAAE;;8DACY"}
+{"version":3,"file":"update.vendor.city.status.dto.js","sourceRoot":"","sources":["../../../../../../../libs/shared/src/lib/data-transfer-objects/src/update.vendor.city.status.dto.ts"],"names":[],"mappings":";AAAA,OAAO,EACN,MAAM,EACN,UAAU,EACV,UAAU,EACV,QAAQ,EACR,MAAM,EACN,SAAS,EACT,UAAU,GACV,MAAM,iBAAiB,CAAC;AACzB,OAAO,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAE9C,OAAO,EAAE,aAAa,EAAE,MAAM,uBAAuB,CAAC;AAEtD,kDAAkD;AAClD,MAAM,CAAC,MAAM,oCAAoC,GAAG,GAAG,CAAC;AAExD;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,MAAM,OAAO,yBAAyB;CAqCrC;AAnCA;IADC,MAAM,EAAE;;kEACe;AAGxB;IADC,MAAM,EAAE;;yDACM;AASf;IAFC,QAAQ,EAAE;IACV,UAAU,CAAC,EAAE,OAAO,EAAE,6BAA6B,EAAE,CAAC;;+DAClC;AAGrB;IADC,MAAM,CAAC,aAAa,CAAC;;yDACA;AAetB;IAZC,UAAU,CACV,CAAC,GAA8B,EAAE,EAAE,CAClC,GAAG,CAAC,MAAM,KAAK,aAAa,CAAC,QAAQ,CACtC;IACA,QAAQ,EAAE;IACV,UAAU,CAAC,EAAE,OAAO,EAAE,8CAA8C,EAAE,CAAC;IACvE,SAAS,CAAC,oCAAoC,CAAC;IAChD,2EAA2E;IAC3E,2EAA2E;IAC3E,sEAAsE;IACtE,kEAAkE;;IACjE,SAAS,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;;yDACzD;AAIhB;IAFC,UAAU,EAAE;IACZ,MAAM,EAAE;;8DACY"}