fix(wpgraphql-webhooks): resolve installation failure from naming inc…#386
Conversation
|
There was a problem hiding this comment.
Pull Request Overview
This PR fixes installation failures for the wpgraphql-webhooks plugin by standardizing naming conventions across the codebase. The main issue was inconsistent naming between wp-graphql-webhooks and wpgraphql-webhooks which caused installation problems.
- Updates text domain from
wpgraphql-webhookstographql-webhooksfor i18n consistency - Standardizes package naming to
wpgraphql-webhooks(removing hyphens) across composer configurations - Updates documentation and script examples to reflect the corrected naming
Reviewed Changes
Copilot reviewed 7 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| plugins/wpgraphql-webhooks/wpgraphql-webhooks.php | Updates text domain for i18n |
| plugins/wpgraphql-webhooks/src/Admin/WebhooksListTable.php | Updates all i18n text domain references |
| plugins/wpgraphql-webhooks/composer.json | Corrects archive name to match package naming |
| plugins/composer-packages.json | Updates package name and distribution URLs across all versions |
| plugins/README.md | Updates plugin reference in documentation table |
| docs/how-to/install-toolkit-plugins/index.md | Updates composer package names in installation examples |
| .github/scripts/update_composer_package.sh | Updates example URLs in script documentation |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Replaces the use of jq for generating a JSON array of valid plugins with a pure Bash implementation. This improves portability and removes the need for jq in the script's environment.
📦 Plugin Artifacts Ready!Download from GitHub Actions run Available plugins:
See the "Artifacts" section at the bottom of the Actions run page |
|
Thanks @josephfusco for this ❤️ |
Adjustments to #362