Skip to content

Remove pagination from Deployment Frequency API

Nathan Friend requested to merge nfriend-remove-pagination-from-deployment-frequency-api into master

What does this MR do?

Removes pagination from the Deployment Frequency API (developed in !48265 (merged)).

Why?

This endpoint provides statistics about the number of deployments for a given environment over time. By paginating the results, we only retrieve the first 20 data points, not all of the requested data points. This makes it difficult to use this endpoint to render graphs (see screenshots below).

We could fix this by updating the frontend to repeatedly make network requests until it has fetched every page, but this would add quite a bit of complexity to the frontend and would result in a much slower page load.

This endpoint already limits its results to the last 91 days, so the maximum number of elements in any valid dataset is 91. Because of this, I don't think removing pagination is a performance risk.

Screenshots

Before:

before

After:

after

Related to #275991 (closed)

Edited by Nathan Friend

Merge request reports