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

Pw 3311 seed data update #251

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

somethvictory
Copy link
Contributor

Refer to the issue id if any. Include the link to the issue. E.g:

Seed data for all tables

Describe the changes made. What does this PR changes that might be critical. If any critical decisions have been made, make sure you explain the rationale for these decisions.

  • Add seed data for all table
  • Update Factory for all table
  • Add a guard before creating new tenants in all tenants.seeds.rb so that rake db:seed can be run multiple times without conflicting

How does the implementation addresses the problem

After merging this, all tables should have initial data when running rake db:seed.

@somethvictory somethvictory added the WIP Pull request still in a work in progress. Comments are welcome label Jan 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Pull request still in a work in progress. Comments are welcome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants