Skip to content

Make monitoring confirmation messages translatable - #284

Open
yeager wants to merge 1 commit into
Cacti:developfrom
yeager:fix-localized-monitor-actions
Open

yeager wants to merge 1 commit into
Cacti:developfrom
yeager:fix-localized-monitor-actions

Conversation

@yeager

@yeager yeager commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

The bulk monitoring confirmation currently inserts the English words “enable” or “disable” into a translated sentence. This leaves those words untranslated and prevents languages from choosing an appropriate sentence structure.

Make each confirmation a complete translatable message. The selected devices and action remain unchanged.

Validation: xgettext extracts both new messages, git diff --check passes, and Gitleaks found no secrets in the changed file. PHP is unavailable locally, so PHP lint and runtime behavior were not checked locally.

@sonarqubecloud

Copy link
Copy Markdown

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