Add resource-level meta docs. Update top-level meta docs. #1566
Conversation
| @@ -81,8 +81,12 @@ PR please :) | |||
|
|
|||
| #### meta | |||
|
|
|||
| If you want a `meta` attribute in your response, specify it in the `render` | |||
| call: | |||
| A `meta` member can be used to include non-standard meta-information. `meta` can | |||
There was a problem hiding this comment.
s/non-standard/custom (or something to that effect)
There was a problem hiding this comment.
Sure, although that's copy from the JSON API docs.
There was a problem hiding this comment.
You're right. I saw that later :embarrassed:
On Wed, Mar 9, 2016 at 8:40 AM Ben Mills notifications@github.com wrote:
In docs/general/rendering.md
#1566 (comment)
:@@ -81,8 +81,12 @@ PR please :)
meta
-If you want a
metaattribute in your response, specify it in therender
-call:
+Ametamember can be used to include non-standard meta-information.metacanSure, although that's copy from the JSON API docs.
—
Reply to this email directly or view it on GitHub
https://github.com/rails-api/active_model_serializers/pull/1566/files#r55526872
.
|
LGTM, i'll merge, unless @bf4 has actionable comments |
|
Haven't had time to review. I trust it's an improvement, which is the most important thing. First, do no harm and all. B mobile phone
|
|
looks good then, further changes, could be in an additional PR. :-) |
Add resource-level meta docs. Update top-level meta docs.
Purpose
Improve meta documentation.
Changes
Related GitHub issues
#1235