WordPress.org
Get WordPress

Make WordPress Core

Opened 8 months ago

Last modified 2 months ago

#63982 assigned defect (bug)

Comment REST API route: add support multiple statuses

Reported by: adamsilverstein's profile adamsilverstein Owned by: adamsilverstein's profile adamsilverstein
Milestone: 7.1 Priority: normal
Severity: normal Version: 5.5
Component: REST API Keywords: has-patch has-unit-tests needs-refresh
Focuses: Cc:

Description

## Description

  • Add support for multiple statuses (an array) to the comments REST API endpoint.
  • The WP_Comment_Query class already supports this, so the only change needed is to allow this parameter to be a string or an array
  • Some unit tests for this would be nice

Change History (6)

#1 @adamsilverstein
8 months ago

  • Component changed from General to REST API

#3 @wildworks
6 months ago

  • Keywords needs-refresh added
  • Milestone changed from 6.9 to 7.0

Since Beta3 is coming soon, I'd like to punt this ticket to 7.0. At the very least, we need to resolve the conflicts in the attached pull request.

This ticket was mentioned in ​Slack in #core by audrasjb. ​View the logs.
3 months ago

#5 @audrasjb
3 months ago

As per today's bug scrub:
This ticket has a PR with unit tests, but it needs to be refreshed to eliminate conflicts.
Also the last comment in the PR suggests that this may be deprioritized. Pinging @adamsilverstein for more info/status on this ticket :)

#6 @adamsilverstein
2 months ago

  • Milestone changed from 7.0 to 7.1

This isn't quite ready for 7.0.

Note: See TracTickets for help on using tickets.