fixed parameters query param for locations, countries, and providers (#…

…306) * fixed parameters query param for locations, countries and providers * Made requested changes to implement ParametersQuery --------- Co-authored-by: Gabriel Fosse <[email protected]>
openaq · Dec 19, 2023 · a96d9ed · a96d9ed
1 parent 0801197
commit a96d9ed
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -7,4 +7,6 @@ cdk.out/
 *.egg-info
 .build
 openaq_api/openaq_api/templates/*
-openaq_api/openaq_api/static/assets/*
+openaq_api/openaq_api/static/assets/*
+.python-version
+Pipfile
diff --git a/openaq_api/openaq_api/v3/models/queries.py b/openaq_api/openaq_api/v3/models/queries.py
@@ -270,7 +270,7 @@ class Paging(QueryBaseModel):
 
     def pagination(self) -> str:
         return "LIMIT :limit OFFSET :offset"
-
+    
 
 class ParametersQuery(QueryBaseModel):
     """Pydantic query model for the parameters query parameter
@@ -289,7 +289,7 @@ class ParametersQuery(QueryBaseModel):
     def where(self) -> str | None:
         """ """
         if self.has("parameters_id"):
-            return "parameters_id = ANY (:parameters_id)"
+            return "parameter_ids && :parameters_id"
 
 
 class MobileQuery(QueryBaseModel):

diff --git a/openaq_api/openaq_api/v3/routers/locations.py b/openaq_api/openaq_api/v3/routers/locations.py
@@ -12,6 +12,7 @@
     MonitorQuery,
     OwnerQuery,
     Paging,
+    ParametersQuery,
     ProviderQuery,
     QueryBaseModel,
     QueryBuilder,
@@ -69,6 +70,7 @@ class LocationsQueries(
     Paging,
     RadiusQuery,
     BboxQuery,
+    ParametersQuery,
     ProviderQuery,
     OwnerQuery,
     CountryIdQuery,