Allow templates with alternate mime types #120
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Trying to use jbuilder with an alternative mime type (eg,
application/hal+json
) does not work for partial templates because jbuilder forcibly sets the format to:json
in the#partial!
method. This is the error I get:I make a request with
Accept: application/hal+json
, Rails finds theaccounts/index.hal.jbuilder
template, and loads it correctly. That template uses the "account" partial, but because the#partial!
method sets the format to:json
, Rails tries to find aaccounts/_account.json.jbuilder
template, while I have it namedaccounts/_account.hal.jbuilder
.This change just removes setting the format. I'm not sure the original purpose of it, and all the tests pass without it.