Skip to content
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

Use string enum columns instead of integer enum columns #3937

Merged
merged 3 commits into from
Sep 10, 2024
Merged

Use string enum columns instead of integer enum columns #3937

merged 3 commits into from
Sep 10, 2024

Conversation

veganstraightedge
Copy link
Contributor

@veganstraightedge veganstraightedge commented Sep 10, 2024 •
edited
Loading

Continuing:

this PR does 3, 4, and 5 from #3925

  1. ...
  2. ...
  3. move references from the enums to the new enums
  4. drop the old column and old enums
  5. rename new enums and columns back to what they were before (remove temp_ prefix and prefix: true)

@veganstraightedge veganstraightedge merged commit 9fcbcab into main Sep 10, 2024
7 checks passed
@veganstraightedge veganstraightedge deleted the enum_use_enums branch September 10, 2024 05:38
@veganstraightedge veganstraightedge mentioned this pull request Sep 10, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@astronaut-wannabe astronaut-wannabe Awaiting requested review from astronaut-wannabe astronaut-wannabe is a code owner

@bookis bookis Awaiting requested review from bookis bookis is a code owner

@eliduke eliduke Awaiting requested review from eliduke eliduke is a code owner

@evanphx evanphx Awaiting requested review from evanphx evanphx is a code owner

@guitsaru guitsaru Awaiting requested review from guitsaru guitsaru is a code owner

@just1602 just1602 Awaiting requested review from just1602 just1602 is a code owner

Assignees
No one assigned
Labels
↔️ migrations
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@veganstraightedge