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.

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

Fix bulk enqueue for unmigrated 'error_event'; add GoodJob.migrated? check method; use custom enum implementation #1011

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

bensheldon
Copy link
Owner

@bensheldon bensheldon commented Jul 17, 2023

Fixes #1010

It's not possible to use the Rails enum because Rails 7.1a introduces a validation check that the column exists: rails/rails#45734

@bensheldon bensheldon force-pushed the migrated_check branch 5 times, most recently from dc8bb86 to 4f7cf8a Compare July 17, 2023 21:26
@bensheldon bensheldon changed the title Fix bulk enqueue for unmigrated 'error_event'; add GoodJob.migrated? check method Fix bulk enqueue for unmigrated 'error_event'; add GoodJob.migrated? check method; use custom enum implementation Jul 17, 2023
@bensheldon bensheldon added the bug Something isn't working label Jul 17, 2023
@bensheldon bensheldon force-pushed the migrated_check branch 5 times, most recently from 8587faf to d217a3e Compare July 17, 2023 22:33
…` check method; use custom enum implementation
@bensheldon bensheldon merged commit a137fed into main Jul 17, 2023
19 checks passed
@bensheldon bensheldon deleted the migrated_check branch July 17, 2023 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

GoodJob::Bulk.enqueue not handling missing migrations
1 participant