Merge pull request #20 from swarmauri/feature/remove_pagination_from_…

…query_params

Feature/remove pagination from query params

pkgs/crouton/crouton/core/sqlalchemy.py

@@ -77,11 +77,17 @@ def __init__(
     def _parse_query_params(self, query_params: Dict[str, Any]) -> Dict[str, Any]:
         """
         Parse query parameters into filters for the database query.
+        Exclude pagination-related parameters like 'skip' and 'limit'.
         """
         filters = {}
         accepted_fields = self.db_model.__table__.columns
-
+
+        # Exclude pagination-related parameters
+        excluded_params = {"skip", "limit"}
+
         for key, value in query_params.items():
+            if key in excluded_params:
+                continue  # Skip pagination parameters
             if key in accepted_fields:
                 column = getattr(self.db_model, key)
                 try:
@@ -94,20 +100,22 @@ def _parse_query_params(self, query_params: Dict[str, Any]) -> Dict[str, Any]:
                     )
             else:
                 raise HTTPException(400, f"Invalid filter field: {key}")
-
+    
         return filters
 
+
     def _get_all(self, *args: Any, **kwargs: Any) -> CALLABLE_LIST:
         def route(
             db: Session = Depends(self.db_func),
             pagination: PAGINATION = self.pagination,
             query_params: Dict[str, Any] = Depends(query_params),
         ) -> List[Model]:
-            skip, limit = pagination.get("skip"), pagination.get("limit")
-
+            skip = pagination.get("skip", 0)
+            limit = pagination.get("limit", 100)  # Default values if not provided
+
             # Parse and validate query parameters
             filters = self._parse_query_params(query_params)
-
+    
             # Apply filters to the query
             db_models: Model = (
                 db.query(self.db_model)
@@ -117,14 +125,15 @@ def route(
                 .offset(skip)
                 .all()
             )
-
+    
             if db_models:
                 return db_models
             else:
                 raise NOT_FOUND from None
-
+    
         return route
 
+
     def _get_one(self, *args: Any, **kwargs: Any) -> CALLABLE:
         def route(
             item_id: self._pk_type, db: Session = Depends(self.db_func)  # type: ignore

pkgs/crouton/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "crouton"
-version = "0.0.4.dev10"
+version = "0.0.4.dev11"
 description = "A repository to enable API CRUD Routing"
 authors = ["Jacob Stewart <[email protected]>"]
 license = "Apache-2.0"

pkgs/crouton_client/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "crouton_client"
-version = "0.0.4.dev10"
+version = "0.0.4.dev11"
 description = "A client for Swarmauri's crouton."
 authors = ["Jacob Stewart <[email protected]>"]
 license = "Apache-2.0"