Skip to content

[WTEL-10355]fix(cc_queue): add default queue value for progressive_count - #490

Merged
suifri merged 1 commit into
mainfrom
fix/WTEL-10355-default-progressive-count
Sep 4, 2026
Merged

suifri merged 1 commit into
mainfrom
fix/WTEL-10355-default-progressive-count

Conversation

@suifri

@suifri suifri commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@webitel-review

webitel-review Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

🤖 Webitel Code Review

Цей коміт змінює поведінку валідації черги (Queue.IsValid()) для типів PredictCall та ProgressiveCall. Замість повернення помилки BadRequestError, якщо параметр progressive_count відсутній або менший/рівний нулю, тепер автоматично встановлюється значення за замовчуванням (1). Також додано ініціалізацію q.Payload, якщо вона була nil, що запобігає паніці при записі в мапу. Ризик змін низький.

📋 Walkthrough (1 файл(и/ів))
Файл Зміни
model/cc_queue.go Додано константу значення за замовчуванням для progressive_count та змінено метод IsValid для автоматичного встановлення цього значення замість повернення помилки.
🔸 Дрібниці / nitpicks (1)
  • [nit] model/cc_queue.go:334 — Друкарська помилка у назві змінної progessiveCount (пропущено літеру 'r' у слові progressive). Це не впливає на роботу коду, але для чистоти варто виправити на progressiveCount.

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

@suifri
suifri force-pushed the fix/WTEL-10355-default-progressive-count branch from 029d582 to 71d921e Compare September 4, 2026 10:30
@suifri
suifri merged commit 2547808 into main Sep 4, 2026
10 of 12 checks passed
@suifri
suifri deleted the fix/WTEL-10355-default-progressive-count branch September 4, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant