Skip to content

Migrate from String Based -> Integer Based Enums #158

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Migrate from String Based -> Integer Based Enums #158

wants to merge 1 commit into from

Conversation

@cjbmartinez
Copy link
Collaborator

Description

Moved to using Integer Based Enums for the following table columns

  • Transcript#process_status
  • SiteAlert#level
  • UserRole#transcribing_role

Changes

  • Create a 3 step migration to be able to successfully update from string to integer based enum, While ensuring all existing records will still have their data intact
  • Minor updates on enum usage and raw SQL Queries using the stirng based enums

neumerance
neumerance reviewed Jul 16, 2025
View reviewed changes
app/models/site_alert.rb

# TODO: Convert to integer-based enum
enum :level, { status: 'status', warning: 'warning', error: 'error' }
enum :level, { status: 0, warning: 1, error: 2 }
Copy link
Collaborator

Choose a reason for hiding this comment

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

nice!

@neumerance
Copy link
Collaborator

PR looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@neumerance neumerance neumerance left review comments

@rtrvrtg rtrvrtg Awaiting requested review from rtrvrtg

@ingnam ingnam Awaiting requested review from ingnam

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cjbmartinez @neumerance