Skip to content

Fix WB-CT309 shop link to point at v.2 - #550

Open
dnnik wants to merge 2 commits into
mainfrom
feature/doc-2130_fix_wb_ct309_link
Open

Fix WB-CT309 shop link to point at v.2#550
dnnik wants to merge 2 commits into
mainfrom
feature/doc-2130_fix_wb_ct309_link

Conversation

@dnnik

@dnnik dnnik commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

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

Два исправления по следам #403 (задача doc-2130), оба — приведение сайта в соответствие с документацией.

1. Ссылка на WB-CT309 вела на первую ревизию. В таблице подбора (catalog/includes/current_transformers_list) строка WB-CT309 (ZMCT123) ссылалась на /product/WB-CT309/, хотя речь про v.2. Исправлено на /product/wb-ct309-v2/ в обеих локалях. Оба адреса отдают 200, поэтому проверка по коду ответа подмену не замечала — она видна только по заголовку страницы.

2. Неверный вес ZEMCTK05-14. На странице товара стояло «вес (без проводов) ~15 г», в документации — «Масса без провода — 60 г». Расхождение в четыре раза. Исправлено на 60 г в обеих локалях, тильда убрана: в документации значение точное.


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

  • клик по WB-CT309 (ZMCT123) в таблице подбора ведёт на карточку v.2, а не v.1;
  • в характеристиках ZEMCTK05-14 указан верный вес.

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

  • pnpm run lint:content — проходит
  • ссылки: прогнал все 11 ссылок таблицы курлом и сверил не код ответа, а <title> страницы — каждая ведёт на карточку своей модели, других подмен ревизий нет. Было Трансформатор тока 5 (125) A, 9 мм, WB-CT309, стало … WB-CT309 v.2. Английские адреса всех 11 моделей тоже отдают 200.
  • вес: сверил список характеристик страницы с таблицей на вики-странице ZEMCTK05-14. Значение 60 г взято из вики; по приложенному к ней даташиту ZEMCTK05-14.pdf не проверял — если есть сомнения, стоит сверить с ним.
  • заодно построчно сверил таблицу подбора с Wbincludes:Current_Transformers_List: все 11 моделей, диапазоны, диаметры и типы сердечника совпадают.

Отрисовку локально не проверял — правки только в тексте, разметка не менялась.


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

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

  • правописание в тексте, опечатки
  • ссылка ведёт на ту ревизию товара, которая описана в строке таблицы
  • вес 60 г подтверждается даташитом, а не только вики

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

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

Риски:

PR содержит два не связанных между собой исправления в разных файлах (таблица-инклюд и страница товара). Разделение на два PR обсуждалось и отклонено автором — оба исправления мелкие и относятся к одной задаче.

Вес 60 г взят из вики без сверки с даташитом производителя.


ИИ: текст подготовил ИИ-агент.
Модель: 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 marked this pull request as ready for review September 3, 2026 12:31
@dnnik
dnnik requested a review from a team as a code owner September 3, 2026 12:31
@dnnik
dnnik requested a review from ElenaSavch September 3, 2026 12:32
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