-
Notifications
You must be signed in to change notification settings - Fork 2
chore: Rename webhooks plugin #291
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
|
ℹ️ Download the latest wpgraphql-webhooks plugin zip from this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR renames the WPGraphQL “headless” webhooks plugin to remove the “headless” term, consolidating the plugin slug and directory name to wpgraphql-webhooks. It also adds Codeception test suite configuration files for unit, functional, acceptance, and WP unit tests.
- Renamed plugin directory and slug from
wp-graphql-webhookstowpgraphql-webhooksand updated references. - Added Codeception test suite configuration files under
tests/. - Updated plugin listing in the root
plugins/README.mdand example path in.wp-env.json.
Reviewed Changes
Copilot reviewed 62 out of 72 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| plugins/wpgraphql-webhooks/wpgraphql-webhooks.php | Renamed plugin slug and header metadata |
| plugins/README.md | Updated plugin listing to use new directory name |
| examples/next/webhooks-isr/.wp-env.json | Updated plugin path in environment config |
| plugins/wpgraphql-webhooks/tests/*.suite.dist.yml | Added Codeception test suite configuration |
Comments suppressed due to low confidence (2)
plugins/wpgraphql-webhooks/wpgraphql-webhooks.php:18
- The header lists
BSD-0-Clausebut the License URI points toBSD-3-Clause; please unify the license name and URI to the correct SPDX identifier.
* License: BSD-0-Clause
plugins/wpgraphql-webhooks/tests/wpunit.suite.dist.yml:1
- [nitpick] You’ve added configuration for the WP unit tests but no actual test files; consider adding at least one basic test case to validate core plugin functionality.
# Codeception Test Suite Configuration
…gine/hwptoolkit into refine-webhooks-plugin-name
|
ℹ️ Download the latest hwp-previews plugin zip from this PR |
|
Isn't every other WPGraphQL plugin named with the |
|
There seems to be a split between some of the ecosystem, but from my understanding we are wanting to move to "wpgraphql" rather than "wp-graphql" https://wordpress.org/plugins/wpgraphql-smart-cache/ |

This pull request renames the webhooks plugin to remove "headless" as it's inferred with WPGraphQL.