Skip to content

[WTEL-10385]feature(cc_queue): add additional max_agent_line validation - #504

Merged
suifri merged 1 commit into
mainfrom
feature/WTEL-10385-queues-max-agent-lines-validation
Sep 11, 2026
Merged

[WTEL-10385]feature(cc_queue): add additional max_agent_line validation#504
suifri merged 1 commit into
mainfrom
feature/WTEL-10385-queues-max-agent-lines-validation

Conversation

@suifri

@suifri suifri commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@webitel-review

Copy link
Copy Markdown
Contributor

🤖 Webitel Code Review

Цей комміт додає підтримку нового параметра конфігурації max_agent_line для прогресивних та предиктивних черг, а також рефакторить валідацію черг у model/cc_queue.go. Проте, зміни впроваджують критичну зворотну несумісність, роблячи поле Payload обов'язковим для всіх типів черг (включаючи вхідні), що може зламати роботу з існуючими даними.

📋 Walkthrough (3 файл(и/ів))
Файл Зміни
model/cc_queue.go Додано константи для ліміту ліній агентів, рефакторизовано метод IsValid та винесено логіку валідації в окремі допоміжні методи.
model/cc_queue_test.go Оновлено та розширено набір юніт-тестів для перевірки нових правил валідації черг та параметрів ліміту ліній.
model/utils.go Додано допоміжний метод IsEmpty для структури Lookup.

Знахідки

  • [high] model/cc_queue.go:329 — Перевірка q.Payload == nil тепер є обов'язковою для всіх типів черг. Раніше для прогресивних/предиктивних черг Payload ініціалізувався автоматично, а для інших (наприклад, InboundCall) міг бути nil. Це критична зворотна несумісність (breaking change), яка призведе до помилок валідації для існуючих черг у базі даних або при API-запитах, де payload не передається. Потрібно повернути автоматичну ініціалізацію Payload лише для прогресивних/предиктивних черг.
🔗 Cross-repo callers змінених символів (1)
  • IsEmptycc-workspaces/packages/electron-sip/sip_client.cpp:230

Index-grounded review across the Webitel codebase. Знахідки можуть бути неточними — перевіряйте перед застосуванням.

@suifri
suifri merged commit b32b751 into main Sep 11, 2026
12 of 13 checks passed
@suifri
suifri deleted the feature/WTEL-10385-queues-max-agent-lines-validation branch September 11, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant