@webitel/api-services 26.8.21 → 26.8.22

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@webitel/api-services",
-	"version": "26.8.21",
+	"version": "26.8.22",
 	"private": false,
 	"repository": {
 		"type": "git",

package/src/validations/queue/__tests__/queue.validations.spec.ts

@@ -174,10 +174,20 @@ describe('queueSchema', () => {
 		});
 	});
 
-	/** Legacy required `maxAttempts` on six types; the backend never did. WTEL-10326 */
-	it.each(allQueueTypes)('lets type %i save without maxAttempts', (type) => {
+	/**
+	 * Dialing numbers the legacy switch required and the backend does not.
+	 * WTEL-10292, WTEL-10326
+	 */
+	const legacyRequiredNumbers = [
+		'payload.maxAttempts',
+		'payload.originateTimeout',
+		'payload.waitBetweenRetries',
+		'payload.maxWaitTime',
+	];
+
+	it.each(allQueueTypes)('lets type %i save with those cleared', (type) => {
 		const queue = validQueueFor(type);
-		set(queue, 'payload.maxAttempts', null);
+		for (const path of legacyRequiredNumbers) set(queue, path, null);
 
 		expect(issuePaths(queueSchema.safeParse(queue))).toEqual([]);
 	});

package/src/validations/queue/queue.rules.ts

@@ -19,10 +19,10 @@ export interface QueueTypeRule {
  *   queues that save fine today.
  * - `priority` and `payload.minOnlineAgents` carry `minValue(0)` for *all* ten
  *   types, including the ones whose form never shows `minOnlineAgents`.
- * - `payload.originateTimeout`, `payload.waitBetweenRetries` (WTEL-10292) and
- *   `payload.maxAttempts` (WTEL-10326) drop the legacy `required`: the backend
- *   takes a queue without them, so it only blocked saving a cleared field.
- *   Their `minValue` stays — it skips an empty value.
+ * - `payload.originateTimeout`, `payload.waitBetweenRetries`, `payload.maxWaitTime`
+ *   (WTEL-10292) and `payload.maxAttempts` (WTEL-10326) drop the legacy
+ *   `required`: the backend takes a queue without them, so it only blocked
+ *   saving a cleared field. Their `minValue` stays — it skips an empty value.
  */
 export const sharedQueueRules: QueueTypeRule = {
 	required: [
@@ -47,7 +47,6 @@ export const queueTypeRules: Record<number, QueueTypeRule> = {
 	[QueueType.INBOUND_QUEUE]: {
 		required: [
 			'payload.timeBaseScore',
-			'payload.maxWaitTime',
 		],
 		minValue: {
 			'payload.maxWaitTime': 0,
@@ -109,7 +108,6 @@ export const queueTypeRules: Record<number, QueueTypeRule> = {
 		required: [
 			'strategy',
 			'payload.timeBaseScore',
-			'payload.maxWaitTime',
 		],
 		minValue: {
 			'payload.maxWaitTime': 0,
@@ -121,7 +119,6 @@ export const queueTypeRules: Record<number, QueueTypeRule> = {
 		required: [
 			'strategy',
 			'payload.timeBaseScore',
-			'payload.maxWaitTime',
 		],
 		minValue: {
 			'payload.maxWaitTime': 0,
@@ -132,7 +129,6 @@ export const queueTypeRules: Record<number, QueueTypeRule> = {
 		required: [
 			'strategy',
 			'calendar',
-			'payload.waitBetweenRetries',
 		],
 		minValue: {
 			'payload.waitBetweenRetries': 0,