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 json.partial!(@record) #265
Comments
Shouldn't be that hard. I can do this over the weekend or you take a shot if you need it sooner :) |
Would love that. Gracias! On Wed, May 13, 2015 at 6:08 PM, Pavel Pravosud notifications@github.com
|
This is done in the https://github.com/rails/jbuilder/tree/ar-partial-api branch. It'd be great if you give it a try before we proceed with documentation and merging. :) |
We'll give it a try! @zachwaugh from Basecamp is working on a bunch of On Tue, May 19, 2015 at 9:53 AM, Pavel Pravosud notifications@github.com
|
Hey @zachwaugh, do you have any feedback? Does everything work for you? |
@rwz Tested it out, and everything looks good here 👍 , thanks! |
@rwz curious if this should also consider
As far as I can tell that doesn't work implicitly at the moment. |
Currently you have to write:
ruby json.partial! 'messages/message', message: @message
Even when all that is guessable from the @message object. Instead, we should do like Action Pack:
ruby json.partial! @message
When we check if the passed object responds to #to_partial_path and then uses that to construct.
The text was updated successfully, but these errors were encountered: