fix(tap-uk-ees): handle plain list response from releases endpoint
All checks were successful
Build and Push Docker Images / Build Backend (FastAPI) (push) Successful in 33s
Build and Push Docker Images / Build Frontend (Next.js) (push) Successful in 1m6s
Build and Push Docker Images / Build Pipeline (Meltano + dbt + Airflow) (push) Successful in 1m45s
Build and Push Docker Images / Trigger Portainer Update (push) Successful in 1s

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Tudor Sitaru
2026-03-30 21:47:14 +01:00
parent 17617137ea
commit 570c2b689e

View File

@@ -34,19 +34,12 @@ def get_content_release_id(publication_slug: str) -> str:
def get_all_release_ids(publication_slug: str) -> list[str]: def get_all_release_ids(publication_slug: str) -> list[str]:
"""Return all release IDs for a publication, newest first.""" """Return all release IDs for a publication, newest first."""
url = f"{CONTENT_API_BASE}/publications/{publication_slug}/releases" url = f"{CONTENT_API_BASE}/publications/{publication_slug}/releases"
ids = [] resp = requests.get(url, timeout=TIMEOUT)
page = 1
while True:
resp = requests.get(url, params={"page": page, "pageSize": 20}, timeout=TIMEOUT)
resp.raise_for_status() resp.raise_for_status()
data = resp.json() data = resp.json()
for r in data.get("results", []): # API returns either a plain list or a paginated object with a "results" key
ids.append(r["id"]) releases = data if isinstance(data, list) else data.get("results", [])
paging = data.get("paging", {}) return [r["id"] for r in releases]
if page >= paging.get("totalPages", 1):
break
page += 1
return ids
def download_release_zip(release_id: str) -> zipfile.ZipFile: def download_release_zip(release_id: str) -> zipfile.ZipFile: