Skip to content

fix(procurement): не показывать «0 KZT» как цену позиции - #844

Merged
ShaerWare merged 1 commit into
mainfrom
local/fix/offer-zero-price-label
Aug 31, 2026
Merged

ShaerWare merged 1 commit into
mainfrom
local/fix/offer-zero-price-label

Conversation

@ShaerWare

Copy link
Copy Markdown
Owner

Summary

~11 тыс. site-офферов синкаются с price = 0 (#841) — это «цены нет в базе», а не бесплатный товар. В блоке реальных позиций, который уходит ассистенту, такие строки печатались как 0 KZT:

- Контактор NC1-2508 25А 2НО+2НЗ 230В 50Гц | арт. 221815 | 0 KZT | в наличии

То есть ассистенту прямым текстом предлагалось назвать клиенту нулевую цену. Условие было price is not None, а 0.0 под него подходит. Теперь такие позиции печатаются как «цена не указана».

Замечено при сквозной проверке #843 на проде.

NEWS

🏷 Ассистент больше не назовёт цену «0 ₸»

По части позиций каталога цена в базу не подтягивается. Раньше такие товары ассистент мог показать с ценой 0 ₸ — теперь честно пишет, что цену нужно уточнить.

Test plan

  • pytest tests/unit — 304 passed (те же 3 падения и 2 ошибки, что и на чистом дереве)
  • ruff check + ruff format --check — чисто
  • После деплоя: позиции без цены в блоке помечены «цена не указана»

Relates to #841

🤖 Generated with Claude Code

~11 тыс. site-офферов синкаются с price=0 (#841) — это «цены нет в
базе», а не бесплатный товар. В блоке реальных позиций такие строки
печатались как «0 KZT», то есть ассистенту прямым текстом предлагалось
назвать клиенту нулевую цену. Теперь — «цена не указана».

## NEWS

🏷 **Ассистент больше не назовёт цену «0 ₸»**

По части позиций каталога цена в базу не подтягивается. Раньше такие
товары ассистент мог показать с ценой 0 ₸ — теперь честно пишет, что
цену нужно уточнить.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@ShaerWare
ShaerWare merged commit 0b57c2b into main Aug 31, 2026
3 checks passed
@ShaerWare
ShaerWare deleted the local/fix/offer-zero-price-label branch August 31, 2026 22:27
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