Simplify deployment filtering logic by removing unnecessary response struct; adjust pagination calculations to match updated logic
This commit is contained in:
@@ -105,22 +105,17 @@ func (s *DeploymentsService) List(ctx context.Context, filter *models.Deployment
|
|||||||
filter.Offset = offset
|
filter.Offset = offset
|
||||||
filter.Limit = limit
|
filter.Limit = limit
|
||||||
|
|
||||||
type response struct {
|
var results []models.Deployment
|
||||||
Results []models.Deployment `json:"results"`
|
if err := s.client.post(ctx, "/deployments/filter", filter, &results); err != nil {
|
||||||
Count int `json:"count"`
|
|
||||||
}
|
|
||||||
|
|
||||||
var resp response
|
|
||||||
if err := s.client.post(ctx, "/deployments/filter", filter, &resp); err != nil {
|
|
||||||
return nil, fmt.Errorf("failed to list deployments: %w", err)
|
return nil, fmt.Errorf("failed to list deployments: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
return &pagination.PaginatedResponse[models.Deployment]{
|
return &pagination.PaginatedResponse[models.Deployment]{
|
||||||
Results: resp.Results,
|
Results: results,
|
||||||
Count: resp.Count,
|
Count: offset + len(results),
|
||||||
Limit: limit,
|
Limit: limit,
|
||||||
Offset: offset,
|
Offset: offset,
|
||||||
HasMore: offset+len(resp.Results) < resp.Count,
|
HasMore: len(results) == limit,
|
||||||
}, nil
|
}, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user