search

Automattic / jetpack

Security, performance, marketing, and design tools — Jetpack is made by WordPress experts to make WP sites safer and faster, and help you grow your traffic.
https://jetpack.com/
Other
1.58k stars 799 forks source link

WordPress.com REST API: allow custom order_by parameters in sites/%24site/posts/ queries #15572

Open dvkch opened 4 years ago

dvkch commented 4 years ago

Is your feature request related to a problem? Please describe.

I am building an app that loads data from my self hosted Wordpress website. I also integrated with the forum provided by bbPress. The app orders topics displayed in a forum according to bbPress _bbp_last_active_time meta, but I can't replicate that behaviour on the API side. I could do it using WP-JSON v2 API but all my app uses the Jetpack API so if I could keep it for all requests it would be easier than managing multiple deserializers.

While I understand bbPress is not edited by you, it seems you do support it a bit, cf https://developer.wordpress.com/2013/04/26/custom-post-type-and-metadata-support-in-the-rest-api/

Describe the solution you'd like

The "easiest" way seems to add the ability to order_by a given meta. This would possibly allow a whole lot of uses as a side effect, and you wouldn't have to think about bbPress inner implementation.

Describe alternatives you've considered

Another way could be to create a specific route that makes use of bbPress API like https://www.buddyboss.com/resources/reference/functions/bbp_forum_query_topic_ids/. But it would mean depending on the plugin which doesn't seem ideal

I've also thought of contacting bbPress authors to sync modified date to _bbp_last_active_time but this is not what that parameter is for

Third option was to extend the jetpack API myself but I'm pretty sure this requires way more understanding than I have and the plugin doesn't have the filters or actions needed to do so.

Additional context

It seems WP_Query is already able to do this https://developer.wordpress.org/reference/classes/wp_query/#order-orderby-parameters

jeherve commented 4 years ago

The app orders topics displayed in a forum according to bbPress _bbp_last_active_time meta, but I can't replicate that behaviour on the API side.

Could you give me an example of the requests you've tried to run, with the parameters you've tried to use, so we can take a closer look?

If you'd rather not post those publicly, you can also contact us here.

Thank you!

dvkch commented 4 years ago

I've reached out to you using the mentioned contact form with all data I've been able to think of

jeherve commented 4 years ago

Closing for now, we'll reply to you soon!

dvkch commented 4 years ago

Hi again,

Is there any news on this issue or is there any way I can help ?

Thanks !

jeherve commented 4 years ago

@dvkch Have you not received a reply to the email you had sent us? You should have gotten a reply within a few days of your initial email, maximum.

If you have more questions, don't hesitate to email us again.

dvkch commented 4 years ago

@jeherve I only received a GitHub notification telling you had closed this ticket and you'd get back to me, but I have not received any further email telling how this ticket was being handled. Is there any news on its status ? I'm open to further discussing this via email if need be.

jeherve commented 4 years ago

Do you think you could send us your email again using the same form? We'll send you our reply again; hopefully this time it will not get lost in transit :)

dvkch commented 4 years ago

@jeherve I have sent a new email hopefully containing at least the same level of details as before. The case ID seems to be [09D4YQ-0P2P]. Thanks for your time :)