Code review comment for lp:~peterm-ubuntu/resource-centre/rest-fix

Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

Great stuff. Although it seems the `id` does not filter the feed by the single post. For example: http://insights.canonicalwebteam.com/wp-json/wp/v2/event?_embed=True&id=6180 returns all the events not just the data for the 6180 event.

I think we need some logic in the event function to return all or a single post if the id exists. Also the events feed takes a long time. I think we need to limit the results.

« Back to merge proposal