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

Add paperclip back to the gemfile so legacy migrations work. #1754

Merged
merged 1 commit into from Jul 25, 2020

Conversation

seanmarcia
Copy link
Member

I had a new contributor complain that running rails db:migrate didn't work and that our migrations were broken.

It turns out that since we used to have paperclip in the application and we had a migration that depended on it we need to put this back to let new folks be able to run the migrations.

This is obviously a short term fix. We'll need to eventually remove the gem and the fix the migrations but this immediately unbreaks things and lets new contributors run migrations as expected.

@auto-comment
Copy link

auto-comment bot commented Jul 24, 2020


Thank you for your pull request!
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.
Best,
@rubyforgood/diaper-team

@giacoelho giacoelho merged commit f2128e2 into master Jul 25, 2020
@giacoelho giacoelho deleted the paperclip branch July 25, 2020 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants