Skip to content

chore(lexicon): EN/RU parity and remove duplicate keys#360

Open
Ibochkarev wants to merge 1 commit into
betafrom
chore/lexicon-parity-348
Open

chore(lexicon): EN/RU parity and remove duplicate keys#360
Ibochkarev wants to merge 1 commit into
betafrom
chore/lexicon-parity-348

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Выравнивает лексиконы менеджера EN↔RU, убирает дубли ключей внутри файлов и выносит ms3_vue_* только в топик vue.

Разведены label и toast/success там, где один ключ нес два смысла (ms3_notification_*, ms3_model_section_*, ms3_customer_email_*). Processor VerifyEmail явно грузит minishop3:customer. В .gitignore добавлен локальный каталог .cursor/.

Тип изменений

  • Исправление бага (non-breaking change)
  • Новая функциональность (non-breaking change)
  • Breaking change (изменение, ломающее обратную совместимость)
  • Рефакторинг (без изменения функциональности)
  • Документация
  • Другое (опишите): tech-debt lexicоns + .gitignore

Связанные Issues

Closes #348

Как это было протестировано?

  • Ручное тестирование
  • Автоматические тесты (PHPStan, ESLint)
  • Тестирование на разных версиях PHP/MODX

Прогон аудита ключей: паритет default/vue/product, 0 дублей во всех топиках, ms3_vue_* только в vue.inc.php. php -l на изменённых PHP. ESLint на NotificationsGrid.vue и ModelFieldsGrid.vue.

Конфигурация тестирования:

  • MiniShop3: ветка chore/lexicon-parity-348
  • MODX: —
  • PHP: локальный php -l

Скриншоты (если применимо)

До После

Чеклист

  • Код соответствует стилю проекта
  • Добавлены/обновлены комментарии в сложных местах
  • Изменения не ломают существующую функциональность
  • Лексиконы добавлены на двух языках (ru/en)
  • PHPStan проходит без новых ошибок
  • ESLint проходит без ошибок (для JS/Vue изменений)
  • Обновлён CHANGELOG.md (для значимых изменений)

Дополнительные заметки

После обновления пакета на живой установке стоит пересобрать/переустановить топики лексикона (или очистить кэш lexicon), чтобы новые ключи (ms3_notification_was_enabled, ms3_model_section_now_hidden, ms3_customer_email_verify_success) попали в modLexiconEntry.

Сообщение API после verify email теперь явная success-строка, а не короткий badge «Verified» (раньше last-write-wins). RU toast warning канонизирован как «Предупреждение» (раньше всплывало последнее «Внимание»).

Align manager lexicon topics, move ms3_vue_* exclusively to vue,
split label vs toast keys, and ignore local .cursor/ config.
@Ibochkarev Ibochkarev requested a review from biz87 July 15, 2026 18:35
@Ibochkarev Ibochkarev changed the title chore(lexicon): EN/RU parity and remove duplicate keys (#348) chore(lexicon): EN/RU parity and remove duplicate keys Jul 16, 2026
@Ibochkarev Ibochkarev added priority: low Низкий приоритет, когда будет время enhancement New feature or request tech-debt Maintainability / refactor / architecture debt and removed priority: low Низкий приоритет, когда будет время labels Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request tech-debt Maintainability / refactor / architecture debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Lexicon: паритет EN/RU и уборка дублей ms3_vue_* / ключей

1 participant