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 commercial_entity to accounts #111

Closed
wants to merge 1 commit into from

Conversation

chrp
Copy link

@chrp chrp commented Jan 10, 2019

This adds a polymorphic association named commercial_entity to accounts like the existing commercial_documents exists for entries. This allows the engine to be better separated from the other models in my projects.

  • updated initial migration
  • run rails g plutus:add_commercial_entity_upgrade to upgrade
  • added one trivial spec as required

- updated initial migration
- run rails g plutus:add_commercial_entity_upgrade to upgrade
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 94.47% when pulling 02f530c on chrp:chrp_business_entity into 00a98af on mbulat:master.

@inopinatus
Copy link

you can surely just use existing tenancy option for this. have you tried a tenant model with polymorphic associations for your app-specific needs?

@chrp chrp closed this Nov 27, 2019
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.

None yet

3 participants