Skip to content

Conversation

@jasonbahl
Copy link
Collaborator

What does this implement/fix? Explain your changes.

This PR adds the following:

  • Documentation for how to Submit a WPGraphQL Extension
  • Adds new Schemas for extensions and single-extension to validate changes to the extensions.json
  • Github Workflow that validates the extensions.json file against schemas/extensions.json to ensure that any changes to the extensions.json file (PRs to the community, etc) in the future are valid.
  • Validate extensions before localizing them for the react app
  • Adds filters for plugin developers to be able to customize the extensions list
    • graphql_get_extensions: Filter the list of extensions
    • graphql_pre_get_extensions: Prevent the JSON file from being loaded in the first place (i.e. override it completely)
    • Documentation for the filters
  • Removed all extensions other than WPGraphQL Smart Cache (will add the other extensions back, one at a time following the new process where we smoke test and document a few things in the PR that adds it).

…ons page works

- add /packages/extensions/README.md file explaining how the extensions react app works
- added a `/schemas` directory with schemas for `extensions.json` and `single-extension.json` and a README.md explaining the schemas
…on adding back more one at a time following the new processes.
…s better. Provided documentation for how to filter extensions (modify / override the list of extensions)
- re-built the react app
@jasonbahl jasonbahl self-assigned this Nov 19, 2024
@jasonbahl jasonbahl changed the title Feat/3205 extensions criteria feat: submit an extension criteria and validation Nov 19, 2024
@justlevine
Copy link
Collaborator

@jasonbahl I opened a PR showing an alternative PHP-only approach in #3246 - for your consideration.

@jasonbahl jasonbahl merged commit 06ee91f into feat/3049-extensions-page-fix Dec 17, 2024
3 checks passed
@justlevine justlevine deleted the feat/3205-extensions-criteria branch December 28, 2024 22:40
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.

3 participants