Skip to content

fix(migrations): converge org nickname schema#8

Merged
rowan-stein merged 1 commit intomainfrom
noa/issue-7
May 1, 2026
Merged

fix(migrations): converge org nickname schema#8
rowan-stein merged 1 commit intomainfrom
noa/issue-7

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • add an idempotent migration to rename org_id to organization_id when needed
  • ensure org_nicknames unique indexes exist for nickname/identity/installation constraints

Testing

  • go test ./...
  • go vet ./...
  • go build ./...

Ref #7

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

  • go test ./... (all packages passed)
  • go vet ./... (no issues)
  • go build ./... (success)

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Migration looks correct and matches #7’s converging/idempotent requirements:

  • renames org_id -> organization_id only when needed
  • (re)ensures the three expected unique indexes via IF NOT EXISTS

Optional: if this PR is intended to close the issue, consider changing the PR text to Fixes #7 instead of Ref #7.

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.

3 participants