-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Extend Decorator documentation #3045
Extend Decorator documentation #3045
Conversation
This change adds an example of using ActiveRecord inside a model decorator. Largely adapted from a gist: https://gist.github.com/kushniryb/101c161eab8d2d2e2e35e4a17b781d1b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments but I'm super 👍 updating documentation since this is something asked often.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jacobherrington !!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 ❤️ 🚀 @jacobherrington thank you for your continuous improvement of Solidus docs!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work
This change adds an example of using ActiveRecord inside a model
decorator. Largely adapted from a gist:
https://gist.github.com/kushniryb/101c161eab8d2d2e2e35e4a17b781d1b