-
-
Notifications
You must be signed in to change notification settings - Fork 6
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 associated object generator #23
Conversation
Run with: `bin/rails g associated Organization::Seats` Co-authored-by: Garrett Dimon <[email protected]>
@garrettdimon thanks for the generator code! I think this is really interesting. I've taken the liberty of tweaking certain names/style to be closer to my mental model. I've cut the integrating "works with associated object" test since I don't quite understand the need for it — imo that would usually be tested through other behavior in the record itself. |
Looks good and much more refined than my first draft. :)
Yeah. That was a more whimsical addition that I started on but never quite felt like it was needed and thus never actually took it any further. Definitely agree with cutting it. |
This is cool! |
Run with:
bin/rails g associated Organization::Seats