Skip to content

fix(schema.rb): remove unbacked columns added by self-check-in migrations - #2776

Merged
mroderick merged 1 commit into
masterfrom
fix/schema-rb-drift
Aug 3, 2026
Merged

fix(schema.rb): remove unbacked columns added by self-check-in migrations#2776
mroderick merged 1 commit into
masterfrom
fix/schema-rb-drift

Conversation

@mroderick

@mroderick mroderick commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

What

Removes check_in_code and source columns/indexes from db/schema.rb that are not backed by any migration file on master.

Why

The only migrations that add these columns live on feature/self-check-in (commit 1ed1a544), which has not been merged to master. Having them in db/schema.rb without corresponding migrations makes the schema inconsistent with db/migrate and breaks the ability to replay migrations from scratch.

This is a prerequisite for the migration-squash work in feature/squash-pre-2026-migrations.

Verification

  • ruby -c db/schema.rb passes.
  • Hiding db/schema.rb and running RAILS_ENV=test TEST_ENV_NUMBER=fixverify bundle exec rails db:create db:migrate reproduces the committed db/schema.rb exactly (diff is empty).

Removed

  • events.check_in_code + unique index
  • workshops.check_in_code + unique index
  • invitations.source
  • workshop_invitations.source

…ions

Removes check_in_code and source columns/indexes that are not defined by any migration on master. The only migrations that add them live on the feature/self-check-in branch (commit 1ed1a54), which has not merged.

This makes db/schema.rb consistent with db/migrate again, which is a prerequisite for the migration squash in feature/squash-pre-2026-migrations.
@mroderick
mroderick marked this pull request as ready for review August 3, 2026 16:07
@mroderick
mroderick merged commit f8bcb95 into master Aug 3, 2026
9 checks passed
@mroderick
mroderick deleted the fix/schema-rb-drift branch August 3, 2026 16:10
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