Merge pull request #1082 from planetlabs/hw/reorder_list_kwargs

Reorder list kwargs to maintain consistency between releases

Author: HenryW95

CHANGES.txt

@@ -1,8 +1,9 @@
-2.12.0 (2024-12-06)
-- Add parameters to the subscriptions list command: --created, --start-time,
+2.12.0 (2024-12-09)
+- Add parameters to the subscriptions list command: --source-type, --created, --start-time,
   --end-time, --updated, --hosting, --name, --name-contains, and --sort-by.
 - Add parameters to the orders list command: --source-type, --name,
   --name-contains, --created-on, --last-modified, --hosting, and --sort-by.
+- Remove the unused limit parameter of SubscriptionsClient.get_results_csv
 
 2.11.0 (2024-10-21)
 

planet/clients/orders.py

@@ -462,16 +462,17 @@ async def wait(self,
 
         return current_state
 
-    async def list_orders(self,
-                          state: Optional[str] = None,
-                          source_type: Optional[str] = None,
-                          name: Optional[str] = None,
-                          name__contains: Optional[str] = None,
-                          created_on: Optional[str] = None,
-                          last_modified: Optional[str] = None,
-                          hosting: Optional[bool] = None,
-                          sort_by: Optional[str] = None,
-                          limit: int = 100) -> AsyncIterator[dict]:
+    async def list_orders(
+            self,
+            state: Optional[str] = None,
+            limit: int = 100,
+            source_type: Optional[str] = None,
+            name: Optional[str] = None,
+            name__contains: Optional[str] = None,
+            created_on: Optional[str] = None,
+            last_modified: Optional[str] = None,
+            hosting: Optional[bool] = None,
+            sort_by: Optional[str] = None) -> AsyncIterator[dict]:
         """Iterate over the list of stored orders.
 
         By default, order descriptions are sorted by creation date with the last created

planet/clients/subscriptions.py

@@ -59,18 +59,19 @@ def __init__(self,
         if self._base_url.endswith('/'):
             self._base_url = self._base_url[:-1]
 
-    async def list_subscriptions(self,
-                                 created: Optional[str] = None,
-                                 end_time: Optional[str] = None,
-                                 hosting: Optional[bool] = None,
-                                 name__contains: Optional[str] = None,
-                                 name: Optional[str] = None,
-                                 source_type: Optional[str] = None,
-                                 start_time: Optional[str] = None,
-                                 status: Optional[Sequence[str]] = None,
-                                 sort_by: Optional[str] = None,
-                                 updated: Optional[str] = None,
-                                 limit: int = 100) -> AsyncIterator[dict]:
+    async def list_subscriptions(
+            self,
+            status: Optional[Sequence[str]] = None,
+            limit: int = 100,
+            created: Optional[str] = None,
+            end_time: Optional[str] = None,
+            hosting: Optional[bool] = None,
+            name__contains: Optional[str] = None,
+            name: Optional[str] = None,
+            source_type: Optional[str] = None,
+            start_time: Optional[str] = None,
+            sort_by: Optional[str] = None,
+            updated: Optional[str] = None) -> AsyncIterator[dict]:
         """Iterate over list of account subscriptions with optional filtering and sorting.
 
         Note: