Skip to content

Route notification images through API content gateway#70

Merged
raymondjacobson merged 1 commit into
mainfrom
codex/notifications-api-content-gateway
Jul 1, 2026
Merged

Route notification images through API content gateway#70
raymondjacobson merged 1 commit into
mainfrom
codex/notifications-api-content-gateway

Conversation

@raymondjacobson

Copy link
Copy Markdown
Member

Summary

  • default notification image URLs to https://api.audius.co instead of a hard-coded creator node
  • stop using stored creator_node_endpoint values for track and playlist email images
  • add NOTIFICATIONS_CONTENT_NODE_ENDPOINT as a notification-specific override
  • update notification mapping expectations to use the API content gateway

Dependency

Testing

  • npm test -w @pedalboard/notifications -- --runInBand src/__tests__/contentNodeEnv.test.ts
  • npm run typecheck -w @pedalboard/notifications
  • git diff --check

Notes

  • The broader notifications mapping/email Jest command still needs local discovery DB env and fails before assertions in populateDB.ts.
  • Remaining creatornode2.audius.co references in archiver/backfill use non-/content APIs and were left for a separate follow-up.

@raymondjacobson raymondjacobson marked this pull request as ready for review June 30, 2026 23:56
@raymondjacobson raymondjacobson merged commit 6f61343 into main Jul 1, 2026
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant