Skip to content

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

Merged
biz87 merged 1 commit into
betafrom
chore/lexicon-parity-348
Jul 29, 2026
Merged

chore(lexicon): EN/RU parity and remove duplicate keys#360
biz87 merged 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 канонизирован как «Предупреждение» (раньше всплывало последнее «Внимание»).

@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
@Ibochkarev
Ibochkarev force-pushed the chore/lexicon-parity-348 branch 5 times, most recently from 5f7f17e to 6c9f3a4 Compare July 29, 2026 02:06
Align manager lexicon topics, move ms3_vue_* exclusively to vue,
split label vs toast keys, and ignore local .cursor/ config.
@Ibochkarev
Ibochkarev force-pushed the chore/lexicon-parity-348 branch from 6c9f3a4 to 594cb48 Compare July 29, 2026 11:35
@biz87
biz87 merged commit 423ced6 into beta Jul 29, 2026
3 checks passed
@biz87
biz87 deleted the chore/lexicon-parity-348 branch July 29, 2026 16:27
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_* / ключей

2 participants