Skip to content

[WTEL-10385]refactor(cc_queue): add default value assignment on nil - #505

Merged
suifri merged 1 commit into
mainfrom
refactor/WTEL-10385-default-value-on-unsupported-type
Sep 11, 2026
Merged

[WTEL-10385]refactor(cc_queue): add default value assignment on nil#505
suifri merged 1 commit into
mainfrom
refactor/WTEL-10385-default-value-on-unsupported-type

Conversation

@suifri

@suifri suifri commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

input for max_agent_line

@suifri
suifri merged commit d2259eb into main Sep 11, 2026
5 checks passed
@webitel-review

Copy link
Copy Markdown
Contributor

🤖 Webitel Code Review

Цей комміт змінює поведінку валідації параметрів черги progressiveCount та maxAgentLines. Замість повернення помилки при невдалому парсингу значень з Payload, тепер використовуються значення за замовчуванням (QueueDefaultProgressiveCountValue та QueueDefaultMaxAgentLine). Це підвищує стійкість системи до некоректних конфігурацій у Payload.

📋 Walkthrough (1 файл(и/ів))
Файл Зміни
model/cc_queue.go Змінено обробку помилок при отриманні числових параметрів конфігурації черги на використання значень за замовчуванням.
🔸 Дрібниці / nitpicks (1)
  • [nit] model/cc_queue.go:421 — Для кращої узгодженості коду з методом prepareProgressiveCount, у методі validateMaxAgentLines можна застосувати такий самий підхід: замість раннього повернення return nil просто присвоїти дефолтне значення змінній maxAgentLines і дозволити коду виконатися далі.

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

@suifri
suifri deleted the refactor/WTEL-10385-default-value-on-unsupported-type branch September 11, 2026 16:38
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