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

Handling archiveless posts in REST API queries #65

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

renatonascalves
Copy link
Contributor

  • Filters to all post types with REST support;
  • Filter the REST API query with the edit context to include archiveless posts.

fixes #61

@renatonascalves renatonascalves self-assigned this Nov 10, 2023
Copy link
Member

@attackant attackant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple suggestions, no blockers 🐙

@@ -224,15 +224,46 @@ public function updated_post_meta( $meta_id, $object_id, $meta_key, $meta_value
}

/**
* Add a filter to all post types for REST response modification.
* Filters to all post types with REST support.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe should read "Add filters..."

inc/class-archiveless.php Show resolved Hide resolved
@renatonascalves renatonascalves merged commit ef04870 into main Nov 13, 2023
4 checks passed
@renatonascalves renatonascalves deleted the feature/rest-api-support branch November 13, 2023 19:00
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.

Handling archiveless posts in REST API queries
2 participants