-
Notifications
You must be signed in to change notification settings - Fork 2
feat: initial wpgraphql-debug-extensions plugin boilerplate #339
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-debug-extensions plugin zip from this PR |
|
ℹ️ Download the latest wp-graphql-webhooks plugin zip from this PR |
colinmurphy
left a comment
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.
This looks good to me. Some minor suggestions in the PR ❤️ 🚀
plugins/wpgraphql-debug-extensions/wpgraphql-debug-extensions.php
Outdated
Show resolved
Hide resolved
Co-authored-by: Colin Murphy <colinmurphy@users.noreply.github.com>
colinmurphy
left a comment
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.
LGTM 🚀 🚀 🚀
Description
This PR adds the basic plugin skeleton for
wpgraphql-debug-extensions. This includes:Namespace: The primary namespace is
WPGraphQL\Debug.Text Domain: The plugin's text domain has been set to
wpgraphql-debug-extensions.README: The
README.mdfile has been added to describe the plugin's purpose as a base for debug extensions, clarifying that it's currently a skeleton with no functionality beyond the core setup.Complexity Metric: Calculates a "complexity score" for each GraphQL query based on the number of fields, fragments, and inline fragments selected. Provides contextual notes in the debugExtensions output, categorizing queries as "Low," "Moderate," "High," or "Very High" complexity.
Related Issue
#337
Dependant PRs
Type of Change
How Has This Been Tested?
Screenshots
N/A - This PR does not introduce any UI/UX changes.
Checklist