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 name of field in seeds #352

Merged
merged 1 commit into from
Oct 3, 2023
Merged

Fix name of field in seeds #352

merged 1 commit into from
Oct 3, 2023

Conversation

ania-hm
Copy link
Contributor

@ania-hm ania-hm commented Oct 3, 2023

Pull Request Summary

When we deploy the application to fly.io, we need to load seeds every time.
I'm getting information about aborted loading of seeds because of incompatibility name of fields during rails db:seed.

Description of the Changes

Among other things, we changed the names of fields in the database for the person model. We changed blog to blog_url and website to website_url.
But we didn't change the names of the fields for assign_attributes in seeds. Making these changes should fix the problem.

Feedback

N/A

UI Changes

N/A

@torrocus
Copy link
Member

torrocus commented Oct 3, 2023

Well caught!

Among other things,
we changed the names of fields in the database for the person model.
We changed blog to blog_url and website to website_url.

But we didn't change the names of the fields
for assign_attributes in seeds.
The problem occurs when we set up a new application environment.

I'm getting information about aborted loading of seeds
because of incompatibility name of fields during `rails db:seed`.
@ania-hm ania-hm force-pushed the ab-fix-field-name-in-seeds branch from b5bc824 to a1f6b47 Compare October 3, 2023 07:12
@ania-hm ania-hm merged commit b2b356d into main Oct 3, 2023
@ania-hm ania-hm deleted the ab-fix-field-name-in-seeds branch October 3, 2023 07:15
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.

2 participants