Skip to content

Fix text domain mismatches, update hook names, and remove disallowed files#84

Merged
happyloa merged 2 commits intomainfrom
codex/fix-text-domain-mismatch-errors
Dec 21, 2025
Merged

Fix text domain mismatches, update hook names, and remove disallowed files#84
happyloa merged 2 commits intomainfrom
codex/fix-text-domain-mismatch-errors

Conversation

@happyloa
Copy link
Owner

Motivation

  • Resolve numerous Text Domain mismatches reported by the plugin-check by aligning translations to a single domain.
  • Prevent plugin-check naming violations for filter/hook names by adding the plugin-specific prefix.
  • Remove hidden/application files that the plugin-check flagged as not permitted in release artifacts.

Description

  • Replaced the old text domain 'site-add-on-watchdog' with 'Site-Add-on-Watchdog-main' across PHP templates and source files (including templates/*.php, src/*.php, and site-add-on-watchdog.php).
  • Renamed admin filter/hook identifiers from site_add_on_watchdog_admin_* to site_add_on_watchdog_main_admin_* to satisfy the prefix convention (e.g., site_add_on_watchdog_main_admin_risks_per_page).
  • Deleted files flagged by the checks: .gitignore, .distignore, build/package.sh, and phpunit.xml.dist.
  • Updated various admin strings, REST messages, and notification-related i18n calls to use the new text domain and adjusted related usages where necessary.

Testing

  • No automated tests were executed as part of this change.
  • Repository was scanned with rg and files were reviewed to confirm text domain and hook name replacements were applied.
  • Changes were committed with the message Fix plugin check text domain and hook names.
  • No runtime verification or CI steps were run in this rollout.

Codex Task

@happyloa happyloa merged commit f0e6d6a into main Dec 21, 2025
2 checks passed
@happyloa happyloa deleted the codex/fix-text-domain-mismatch-errors branch December 21, 2025 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant