Code review comment for lp:~wallyworld/launchpad/recipe-webservice-api

Revision history for this message
Leonard Richardson (leonardr) wrote :

This looks great now. My only comment is that "sorted by 'desc(date_created), then by desc(id).'" is a bad way to describe a field. This description will be going into the HTML documentation at https://launchpad.net/+apidoc/devel.html, so you should say something understandable by external developers. Something like "Sorted in descending order of creation."

review: Approve

« Back to merge proposal