Ensure that devices are searchable by the presence of a postprocessing

fablabbcn / smartcitizen-api

The Smart Citizen Engine

https://developer.smartcitizen.me

GNU Affero General Public License v3.0

10 stars 4 forks source link

Ensure that devices are searchable by the presence of a postprocessing #255

Closed timcowlishaw closed 10 months ago

timcowlishaw commented 10 months ago

Fixes #254 - this is an API change in the new version of Ransack - searchable parameters have to be explicitly whitelisted. We probably need to investigate to see if there are any others that are used habitually, in the meantime this will fix the immediate issue with postprocessings.

oscgonfer commented 10 months ago

Hi @timcowlishaw I will merge the PR for the ransack and postprocessing, but we need to keep at least these ones:

in the docs: https://developer.smartcitizen.me/#basic-searching
In the python package: https://github.com/fablabbcn/smartcitizen-data/blob/master/scdata/io/device_api.py#L244-L254