Skip to content
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

Patterns API: Omit the "featured" category in the REST API #704

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

ryelle
Copy link
Contributor

@ryelle ryelle commented Jul 22, 2024

This removes the "featured" category from the API response, so that patterns remotely loaded from the pattern directory will never be included in the "Featured" section of local WordPress sites. This breaks the connection between the Pattern Directory "Featured" category & the "Featured" section of the pattern inserter. Patterns in the featured category on the Pattern Directory will still be sent to WordPress sites, and they will still appear the other categories they have.

For example, Simple call to action has categories "Featured, Buttons", so it will only appear in "Buttons" on local WordPress sites.

Fixes #612.
Props richtabor

How to test the changes in this Pull Request:

  1. View an API response /wp-json/wp/v2/wporg-pattern/
  2. None of the patterns should have "featured" in category_slugs

For easier testing, find the ID for "featured" and use that for the API request, /wp-json/wp/v2/wporg-pattern/?pattern-categories=CAT_ID or use slugs for individual patterns, /wp-json/wp/v2/wporg-pattern/?slug=offset-images-with-descriptions.

There should be no change for non-featured patterns.

@ryelle ryelle added the [Component] Pattern Directory API The pattern API on WordPress.org, and/or the CPT endpoint label Jul 22, 2024
@ryelle ryelle self-assigned this Jul 22, 2024
@richtabor
Copy link
Member

Very nice!

@ryelle ryelle force-pushed the remove/featured-cat-api-response branch from e940f4b to 45ac6ea Compare September 30, 2024 18:28
@ryelle ryelle merged commit 060ed90 into trunk Sep 30, 2024
3 checks passed
@ryelle ryelle deleted the remove/featured-cat-api-response branch September 30, 2024 19:58
ParhamG added a commit to ParhamG/pattern-directory that referenced this pull request Oct 1, 2024
This fixes a regression caused via WordPress#704 that returns category_slugs as an object instead of an array
ryelle pushed a commit that referenced this pull request Oct 1, 2024
This fixes a regression caused via #704 that returns category_slugs as an object instead of an array
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Pattern Directory API The pattern API on WordPress.org, and/or the CPT endpoint
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Proposal: Don't ship core patterns to WordPress sites in the "Featured" category
2 participants