Skip to content

Commit

Permalink
set max on listing app strategies
Browse files Browse the repository at this point in the history
  • Loading branch information
IrinaSouth committed Dec 16, 2024
1 parent c8efc0d commit 8062be8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,12 @@ class ApplicationStrategyBloc implements Bloc, ManagementRepositoryAwareBloc {
Future<ApplicationRolloutStrategyList> getStrategiesData(
String? s, SortOrder sortOrder) async {
if (appId != null) {
return await _applicationRolloutStrategyServiceApi
.listApplicationStrategies(appId!,
includeUsage: true, includeWhoChanged: true);
return await _applicationRolloutStrategyServiceApi.listApplicationStrategies(
appId!,
includeUsage: true,
includeWhoChanged: true,
max:
1000); // set max because it defaults to 20 otherwise and we do not use pagination yet
} else {
return ApplicationRolloutStrategyList(max: 0, page: 0, items: []);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ class ApplicationStrategyDataTableSource
Text(strategy.updatedBy.email),
),
DataCell(Text(
'environments: ${strategy.strategy.usage!.length}, feature values: ${strategy.strategy.usage!.map((e) => e.featuresCount).sum}')),
'environments: ${strategy.usage!.length}, feature values: ${strategy.usage!.map((e) => e.featuresCount).sum}')),
DataCell(Row(children: <Widget>[
FHIconButton(
icon: const Icon(Icons.edit),
Expand Down

0 comments on commit 8062be8

Please sign in to comment.