Set page_token explicitly in pagination params to match updated frequenz-client-common

- Set page_token to empty string when not provided, as it is no longer optional
- Affects list methods for gridpool orders, trades, and public trades

Signed-off-by: camille-bouvy-frequenz <[email protected]>

Author: camille-bouvy-frequenz

src/frequenz/client/electricity_trading/_client.py

@@ -804,6 +804,7 @@ async def list_gridpool_orders(
         delivery_area: DeliveryArea | None = None,
         tag: str | None = None,
         page_size: int | None = None,
+        page_token: str | None = None,
         timeout: timedelta | None = None,
     ) -> AsyncIterator[OrderDetail]:
         """
@@ -817,6 +818,7 @@ async def list_gridpool_orders(
             delivery_area: The delivery area to filter by.
             tag: The tag to filter by.
             page_size: The number of orders to return per page.
+            page_token: The token identifying a specific page of the listed orders.
             timeout: Timeout duration, defaults to None.
 
         Yields:
@@ -832,13 +834,17 @@ async def list_gridpool_orders(
             delivery_area=delivery_area,
             tag=tag,
         )
+        pagination_params = None
+        if page_size is not None:
+            pagination_params = Params(
+                page_size=page_size,
+                page_token=page_token if page_token is not None else "",
+            ).to_proto()
 
         request = electricity_trading_pb2.ListGridpoolOrdersRequest(
             gridpool_id=gridpool_id,
             filter=gridpool_order_filter.to_pb(),
-            pagination_params=(
-                Params(page_size=page_size).to_proto() if page_size else None
-            ),
+            pagination_params=pagination_params,
         )
         while True:
             try:
@@ -869,7 +875,7 @@ async def list_gridpool_orders(
                 raise
 
     async def list_gridpool_trades(
-        # pylint: disable=too-many-arguments, too-many-positional-arguments
+        # pylint: disable=too-many-arguments, too-many-positional-arguments, too-many-locals
         self,
         gridpool_id: int,
         trade_states: list[TradeState] | None = None,
@@ -878,6 +884,7 @@ async def list_gridpool_trades(
         delivery_period: DeliveryPeriod | None = None,
         delivery_area: DeliveryArea | None = None,
         page_size: int | None = None,
+        page_token: str | None = None,
         timeout: timedelta | None = None,
     ) -> AsyncIterator[Trade]:
         """
@@ -891,6 +898,7 @@ async def list_gridpool_trades(
             delivery_period: The delivery period to filter by.
             delivery_area: The delivery area to filter by.
             page_size: The number of trades to return per page.
+            page_token: The token identifying a specific page of the listed trades.
             timeout: Timeout duration, defaults to None.
 
         Yields:
@@ -906,13 +914,17 @@ async def list_gridpool_trades(
             delivery_period=delivery_period,
             delivery_area=delivery_area,
         )
+        pagination_params = None
+        if page_size is not None:
+            pagination_params = Params(
+                page_size=page_size,
+                page_token=page_token if page_token is not None else "",
+            ).to_proto()
 
         request = electricity_trading_pb2.ListGridpoolTradesRequest(
             gridpool_id=gridpool_id,
             filter=gridpool_trade_filter.to_pb(),
-            pagination_params=(
-                Params(page_size=page_size).to_proto() if page_size else None
-            ),
+            pagination_params=pagination_params,
         )
 
         while True:
@@ -951,6 +963,7 @@ async def list_public_trades(
         buy_delivery_area: DeliveryArea | None = None,
         sell_delivery_area: DeliveryArea | None = None,
         page_size: int | None = None,
+        page_token: str | None = None,
         timeout: timedelta | None = None,
     ) -> AsyncIterator[PublicTrade]:
         """
@@ -962,6 +975,7 @@ async def list_public_trades(
             buy_delivery_area: The buy delivery area to filter by.
             sell_delivery_area: The sell delivery area to filter by.
             page_size: The number of public trades to return per page.
+            page_token: The token identifying a specific page of the listed trades.
             timeout: Timeout duration, defaults to None.
 
         Yields:
@@ -977,11 +991,16 @@ async def list_public_trades(
             sell_delivery_area=sell_delivery_area,
         )
 
+        pagination_params = None
+        if page_size is not None:
+            pagination_params = Params(
+                page_size=page_size,
+                page_token=page_token if page_token is not None else "",
+            ).to_proto()
+
         request = electricity_trading_pb2.ListPublicTradesRequest(
             filter=public_trade_filter.to_pb(),
-            pagination_params=(
-                Params(page_size=page_size).to_proto() if page_size else None
-            ),
+            pagination_params=pagination_params,
         )
 
         while True: