Skip to content

Add ru and en product pages for three current transformers - #559

Merged
dnnik merged 3 commits into
mainfrom
feature/doc-2130_add_four_ct_pages
Sep 8, 2026
Merged

Add ru and en product pages for three current transformers#559
dnnik merged 3 commits into
mainfrom
feature/doc-2130_add_four_ct_pages

Conversation

@dnnik

@dnnik dnnik commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Что происходит; кому и зачем нужно:

Переносим описания трёх трансформаторов тока из бэкенда магазина в гит — по образцу ZEMCTK05-14 (#403) и остальных страниц серии (#554, #555, #556, #557, #558).

Товар Измеряемый ток Iном Коэффициент Диаметр Сердечник
ZEMCTK04-13 0.25…152 А 25 А 1:3000 16 мм феррит
CTSA024 0.35…290 А 35 А 1:5000 24 мм феррит
CTSA035 5…755 А 500 А 1:10000 35 мм сталь

Все три — класс точности 0.5S, разъёмные. Характеристики взяты со страниц вики (ZEMCTK04-13, CTSA024, CTSA035, ZEMCTK09-31G) и сверены с ними программно. Артикулы подтверждены по карточкам магазина — у всех трёх совпадают с моделью.

Изначально в PR был и ZEMCTK09-31G, но он снят с продажи — страницу убрал коммитом Drop ZEMCTK09-31G page: product discontinued. Из общей таблицы подбора его пока не удалял, см. «Риски».

Задача doc-2130


Что поменялось для пользователей:

У трёх трансформаторов появились страницы товара с описанием, характеристиками, графиком токовой погрешности и таблицей подбора трансформатора по току.


Как проверял/а:

  • pnpm run lint:content — проходит
  • все шесть страниц (3 товара × 2 локали) отдают 200, списки характеристик рендерятся по 9 пунктов, таблица подбора на месте
  • check-images.sh по каждому товару — все картинки отдаются с правильным Content-Type, включая _ipx-варианты
  • характеристики сверены с вики программно: измеряемый ток, номинальный ток, коэффициент, диаметр отверстия, материал сердечника, масса, длина провода — расхождений нет

Чеклист ревью:

Перед мержем не забудь проверить:

  • правописание в тексте, опечатки
  • разрешение фотографии ZEMCTK04-13, см. «Риски»

Опционально (но важно для изменений в инфраструктуру):

  • задеплоить на stage, чтобы проверить, что все работает

Риски:

Три товара в одном PR. Отступление от правила «один PR — одна смысловая единица» и от лимита 250–300 строк: здесь 264 строки в 18 файлах. Сделано по прямому указанию автора, чтобы не дробить однотипную работу на отдельные ревью.

Фотография ZEMCTK04-13 низкого разрешения — 483×857, для обложки товара маловато. CTSA024 (1055×1280) и CTSA035 (999×1280) приемлемы. Руководство по стилю требует исходник не меньше 2320 x 3088 px — ни один снимок серии этому не соответствует. Студийных фото для этих товаров нет.

ZEMCTK09-31G остался в таблице подбора. Товар снят с продажи, его страницу из этого PR убрал, но в общем инклюде catalog/includes/current_transformers_list он всё ещё присутствует как одна из 11 моделей. Таблица предлагает выбрать трансформатор, поэтому снятую с продажи модель из неё, видимо, стоит убрать — отдельным PR, файл лежит в main.

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


ИИ: текст подготовил ИИ-агент.
Модель: claude-opus-5[1m] · Harness: Claude Code CLI 2.1.259 · Настройки: permission mode auto, reasoning effort — unknown
Запустил и отвечает за содержимое: @dnnik

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@dnnik dnnik changed the title Add ru and en product pages for four current transformers Add ru and en product pages for three current transformers Sep 7, 2026
@dnnik
dnnik marked this pull request as ready for review September 7, 2026 06:52
@dnnik
dnnik requested a review from a team as a code owner September 7, 2026 06:52
ElenaSavch
ElenaSavch previously approved these changes Sep 7, 2026
@dnnik
dnnik merged commit 533a978 into main Sep 8, 2026
3 checks passed
@dnnik
dnnik deleted the feature/doc-2130_add_four_ct_pages branch September 8, 2026 13:29
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.

2 participants