Skip to content

Commit c997827

Browse files
Update frequenz-client-common requirement from <0.3.0,>=0.1.0 to >=0.1.0,<0.4.0 (#115)
This PR follows up on #89 and fixes dependency issues as well as new required changes caused by the update of `frequenz-client-common` from <0.3.0,>=0.1.0 to >=0.1.0,<0.4.0. More specifically, the following dependencies were upgraded: * Bump `grpcio` to >=1.68.1,<2 * Bump `protobuf` to >=5.29.2,<6 And `page_token` was added as a parameter to the list functions, so that it is explicitly in the pagination parameters (as it is no longer an optional parameter in the latest version).
2 parents 6e79968 + d757660 commit c997827

File tree

3 files changed

+14
-7
lines changed

3 files changed

+14
-7
lines changed

RELEASE_NOTES.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@
66

77
## Upgrading
88

9-
<!-- Here goes notes on how to upgrade from previous versions, including deprecations and what they should be replaced with -->
9+
* Updated frequenz-client-common version range to >=0.1.0, <0.4.0
10+
* Upgraded grpcio to >=1.68.1, <2 and protobuf to >=5.29.2, <6 to resolve compatibility issues
1011

1112
## New Features
1213

pyproject.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ dependencies = [
3030
"click >= 8.1.8, < 9",
3131
"entsoe-py >= 0.6.16, < 1",
3232
"frequenz-api-common >= 0.6.3, < 0.7.0",
33-
"grpcio >= 1.66.2, < 2",
33+
"grpcio >= 1.68.1, < 2",
3434
"frequenz-channels >= 1.6.1, < 2",
3535
"frequenz-client-base >= 0.9.0, < 0.10.0",
36-
"frequenz-client-common >= 0.1.0, < 0.3.0",
36+
"frequenz-client-common >= 0.1.0, < 0.4.0",
3737
"frequenz-api-electricity-trading >= 0.2.4, < 0.5",
38-
"protobuf >= 5.28.0, < 6",
38+
"protobuf >= 5.29.2, < 6",
3939
]
4040
dynamic = ["version"]
4141

src/frequenz/client/electricity_trading/_client.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -837,7 +837,9 @@ async def list_gridpool_orders(
837837
gridpool_id=gridpool_id,
838838
filter=gridpool_order_filter.to_pb(),
839839
pagination_params=(
840-
Params(page_size=page_size).to_proto() if page_size else None
840+
Params(page_size=page_size, page_token="").to_proto()
841+
if page_size
842+
else None
841843
),
842844
)
843845
while True:
@@ -911,7 +913,9 @@ async def list_gridpool_trades(
911913
gridpool_id=gridpool_id,
912914
filter=gridpool_trade_filter.to_pb(),
913915
pagination_params=(
914-
Params(page_size=page_size).to_proto() if page_size else None
916+
Params(page_size=page_size, page_token="").to_proto()
917+
if page_size
918+
else None
915919
),
916920
)
917921

@@ -980,7 +984,9 @@ async def list_public_trades(
980984
request = electricity_trading_pb2.ListPublicTradesRequest(
981985
filter=public_trade_filter.to_pb(),
982986
pagination_params=(
983-
Params(page_size=page_size).to_proto() if page_size else None
987+
Params(page_size=page_size, page_token="").to_proto()
988+
if page_size
989+
else None
984990
),
985991
)
986992

0 commit comments

Comments
 (0)