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

Added model classes to config so they are easier to extend. #33

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

Conversation

jonjonw
Copy link

@jonjonw jonjonw commented Jul 9, 2015

Now you can specify the model classes to use in the config file and then write classes to extend them without having use traits and re-write all the relationships. Should work for all models other than the base model.

Jon Wiersma added 3 commits July 9, 2015 15:30
Now you can specify the model classes to use in the config file and then write classes to extend them without having use traits and re-write all the relationships. Should work for all models other than the base model.
@stevebauman
Copy link
Collaborator

Great idea, thanks! I'll just have to write a test first before I merge this in. Will be done shortly.

@cdarken
Copy link

cdarken commented Nov 22, 2016

Would be great for this to be merged, because for instance would be easier to add fillable attributes to some models, like Metric, for easy usage with firstOrCreate.

@kingsloi
Copy link

@cdarken - I agree! @stevebauman, any updates on this?

tr3mulant pushed a commit to tr3mulant/inventory that referenced this pull request Sep 23, 2022
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.

5 participants