Be able to request a subset of columns from PiccoloCRUD's GET endpoint about piccolo_api

[sinisaos]

@dantownsend Can you please close this issue because it's done with latest changes.

[wmshort]

@dantownsend great addition. 👍 What's the relationship between the query string (__fields) and the returned JSON key ('rows' in your example)? Are they meant to match?

[dantownsend]

@wmshort Yeah, they're meant to match. The top level rows key is always the same, but the sub elements should have their keys filtered based on the __fields parameter, if that makes sense.

[wmshort]

@dantownsend definitely. I was just wondering about what seemed like a mismatch between that top level key ("rows") and the parameter "fields" in the query. But obviously the result consists of (only) the specified fields of the fetched rows, so your naming scheme makes good sense. The request is for table rows, constrained by certain parameters. Looking forward to this feature 🙂

[dantownsend]

@wmshort This feature is pretty much done now, except I'm not sure on the naming of the parameter.

What do you think is most suitable? __fields, __columns, __visible_fields?

[wmshort]

Personally I find 'fields' the most intuitive but any of these will be clear enough. Glad this has come together. Nice!

[dantownsend]

@wmshort Thanks - naming stuff is hard! It's good to have a sanity check.

[dantownsend]

Fixed by #98