Skip to content

Conversation

pwseg
Copy link
Contributor

@pwseg pwseg commented Aug 7, 2025

Proposed changes

  • Ran the catalog.
  • Fixed TypeError: Cannot read properties of undefined (reading 'catalog') in sources catalog update.
  • It looks like the Public API changed the response structure from res.data.data.catalog to res.data.sourcesCatalog.
  • Updated pagination access from res.data.data.pagination?.next to res.data.pagination?.next.
  • Script now aligns with what we show in the PAPI documentation + matches destinations pattern.

Merge timing

  • ASAP once approved.

@pwseg pwseg requested a review from forstisabella August 7, 2025 19:13
@pwseg pwseg requested a review from a team as a code owner August 7, 2025 19:13
@pwseg pwseg added the back-end fixes or updates to the site itself, which should not go in release notes label Aug 7, 2025
@pwseg pwseg enabled auto-merge August 7, 2025 19:24
@pwseg pwseg merged commit 6508635 into develop Aug 7, 2025
2 checks passed
@pwseg pwseg deleted the playing-with-the-catalog branch August 7, 2025 19:39
Copy link
Contributor

github-actions bot commented Aug 7, 2025

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end fixes or updates to the site itself, which should not go in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants