Skip to content

chore(api): bump etl to persist user social links (#341)#920

Merged
raymondjacobson merged 1 commit into
mainfrom
chore/bump-etl-social-links
Jun 8, 2026
Merged

chore(api): bump etl to persist user social links (#341)#920
raymondjacobson merged 1 commit into
mainfrom
chore/bump-etl-social-links

Conversation

@raymondjacobson

Copy link
Copy Markdown
Member

Summary

Bumps github.com/OpenAudio/go-openaudio and .../pkg/etl from 7062cd9 (#919) to 4cccb46 to consume go-openaudio #341.

That change fixes the User Create/Update handlers silently dropping instagram_handle, twitter_handle, tiktok_handle, website, and donation — so users editing their profile could not add or change social links. Reported by audius.co/allatonce and audius.co/peaceluis. Production data: 167k+ users have an unverified Instagram handle, confirming these are normal profile fields, not verification-only.

core-indexer runs this vendored audius/api image, so this bump is required to ship the fix.

Changes

  • go.mod / go.sum: both go-openaudio modules → v1.3.1-0.20260608204715-4cccb46f1332

Deploy note — migration 0032 (low risk)

This bump pulls ETL migration 0032, which adds the website + donation columns with IF NOT EXISTS. In prod these columns already exist (inherited legacy schema), so 0032 is a no-op there — no lock concern (unlike the 0031 index build). No special rollout needed.

Test plan

  • go mod tidy — only go.mod/go.sum changed
  • go build ./... passes
  • resolved module contains the social-field writes (instagram_handle in user_update.go) and migration 0032
  • Post-deploy: allatonce / peaceluis can save IG handle + website; verified_with_* unchanged

🤖 Generated with Claude Code

Bumps go-openaudio to 4cccb46, which fixes the User Create/Update
handlers silently dropping instagram_handle/twitter_handle/tiktok_handle/
website/donation, and adds migration 0032 for the website/donation
columns. Unblocks users adding/changing social links on their profile.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@raymondjacobson raymondjacobson merged commit 13417e3 into main Jun 8, 2026
5 checks passed
@raymondjacobson raymondjacobson deleted the chore/bump-etl-social-links branch June 8, 2026 20:56
raymondjacobson added a commit that referenced this pull request Jun 9, 2026
…924)

## Summary

Bumps `github.com/OpenAudio/go-openaudio` and `.../pkg/etl` from
`4cccb46` (#920) to **`c5fcacf`** to consume go-openaudio **#343**.

That change fixes entity-manager handlers silently dropping columns that
exist on the table and arrive in metadata (same class as the
social-links fix, found by auditing all write handlers):
- **Track** create/update now persist `license`, `isrc`, `iswc`,
`preview_start_seconds`, `comments_disabled`, `no_ai_use`,
`cover_original_song_title`, `cover_original_artist`. Prod scale of the
prior loss: 642,291 tracks have a `license`, 94,894 an `isrc`, 31,615 a
custom `preview_start_seconds`.
- **Comment** update now preserves/updates `video_url` (a text-only edit
no longer wipes an attached video).

`core-indexer` runs this vendored `audius/api` image, so this bump ships
the fix.

## Changes
- `go.mod` / `go.sum`: both go-openaudio modules →
`v1.3.1-0.20260609000702-c5fcacffbb79`

## Deploy note — no migration
This is **code-only**: every affected column already exists in migration
`0002`/`0032`, so there's no new migration and no schema/rollout
concern. Latest ETL migration stays `0032`.

## Test plan
- [x] `go mod tidy` — only go.mod/go.sum changed
- [x] `go build ./...` passes
- [x] resolved module contains the track field writes (`License` in
track_row.go) and comment `video_url` COALESCE; latest migration
unchanged (0032)
- [ ] Post-deploy: track license/isrc/preview-start and comment video
edits persist (verify on a test account)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
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