Skip to content

Conversation

@jasonbahl
Copy link
Collaborator

@jasonbahl jasonbahl commented Feb 23, 2024

Release Notes

New Features

  • #3044: feat: add graphql_pre_resolve_menu_item_connected_node filter
  • #3039: feat: add UniformResourceIdentifiable interface to Comment type
  • #3020: feat: introduce graphql_query_analyzer_get_headers filter

Chores / Bugfixes

  • #3062: ci: pin wp-browser to "<3.5" to allow automated tests to run properly
  • #3057: fix: admin_enqueue_scripts callback should expect a possible null value passed to it
  • #3048: fix: isPostsPage on content type
  • #3043: fix: return empty when filtering menuItems by a location with no assigned items
  • #3045: fix: UsersConnectionSearchColumnEnum values should be prefixed with user_

MuhammedAO and others added 30 commits January 19, 2024 17:30
fix: `UsersConnectionSearchColumnEnum` values should be prefixed with `user_`
fix: return empty when filtering `menuItems` by a `location` with no assigned items
feat: add `UniformResourceIdentifiable` interface to `Comment` type
…_item_connected_node

feat: add `graphql_pre_resolve_menu_item_connected_node` filter
…returns a `ContentType` that `isPostsPage` field returns true
…ble Interface as any node with a URI should be able to identify whether it is or is not the posts page or front page. Defaults to false, but individual implementing types (ContentType, Page) can override the resolver as applicable.

- update the PostType Model to return true if show_on_front is set to "page" and update Post model to return `false` for `isPostsPage` as a Post Object won't ever resolve as the posts page based on the current NodeResolver strategy (see: https://github.com/wp-graphql/wp-graphql/blob/develop/src/Data/NodeResolver.php#L226)
…tent-type

fix: isPostsPage on content type
…argument don't throw errors. It's supposed to pass a string, but it's been reported to not always pass a string.
@jasonbahl jasonbahl added the type: release Pull request intended for a release label Feb 23, 2024
@jasonbahl jasonbahl self-assigned this Feb 23, 2024
@coveralls
Copy link

coveralls commented Feb 23, 2024

Coverage Status

coverage: 84.372% (+0.1%) from 84.275%
when pulling 7c48660 on release/v1.22.0
into 8b106a5 on master.

@jasonbahl jasonbahl merged commit 933de8a into master Mar 4, 2024
@justlevine justlevine deleted the release/v1.22.0 branch August 3, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: release Pull request intended for a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants