Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#51458 closed task (blessed) (fixed)

REST API: Add terms search handler

Reported by: timothyblynjacobs's profile TimothyBlynJacobs Owned by: timothyblynjacobs's profile TimothyBlynJacobs
Milestone: 5.6 Priority: normal
Severity: normal Version: 5.0
Component: REST API Keywords: has-patch has-unit-tests
Focuses: Cc:

Description

This is a tracking ticket for merging the Terms search handler from Gutenberg.

https://github.com/WordPress/gutenberg/blob/master/lib/class-wp-rest-term-search-handler.php

Change History (7)

#2 @TimothyBlynJacobs
4 years ago

@noisysocks see #51459.

This ticket was mentioned in PR #577 on WordPress/wordpress-develop by TimothyBJacobs.


4 years ago
#3

  • Keywords has-patch has-unit-tests added

TimothyBJacobs commented on PR #577:


4 years ago
#4

@noisysocks Could you give this a test with Gutenberg active?

noisysocks commented on PR #577:


4 years ago
#5

@noisysocks Could you give this a test with Gutenberg active?

It works 👍

I suspect the handler added by Core is used as it's first in the array. We should update Gutenberg to not add the handler if there is already one.

#6 @TimothyBlynJacobs
4 years ago

  • Owner set to TimothyBlynJacobs
  • Resolution set to fixed
  • Status changed from new to closed

In 49103:

REST API: Introduce search term handler.

This allows for clients to search the available terms via the /wp/v2/search endpoint by using a type=term query parameter.

Fixes #51458.
Props andraganescu, zieladam, noisysocks, TimothyBlynJacobs.

TimothyBJacobs commented on PR #577:


4 years ago
#7

Merged in 5b6f5f9.

Note: See TracTickets for help on using tickets.