Fourth pass at post meta handler - #207

I successfully tested viewing post_meta, adding single value meta keys and values, and editing/adding array values for meta keys.

@tlovett1 would you be able to test this code as well?

Next steps:

• add functionality docs for post_meta
• HATEOAS links

Yes, I'll test.

I tested edit/create/delete. Everything worked well except changing a key without changing the value. Do we have unit tests for this yet?

@rmccue #review 1eee8b9 and 710aac6

Still needed POST /post/ and PUT /post/id post_meta data parameter details. Blocked by merge of #212 that includes data parameter details for those routes.

We're now at 95% minimum code coverage for meta methods; can't test the internal add_post_meta/update_metadata_by_mid/delete_metadata_by_mid responses, since we check for those cases beforehand anyway.

@rachelbaker Can we hit those extra docs in 1.1 instead? If so, let's #reviewmerge and ship 1.0.

Let's add the extra docs in 1.1, I will review right after breakfast.

On Saturday, May 24, 2014, Ryan McCue [email protected] wrote:

We're now at 95% minimum code coverage for meta methods; can't test the
internal add_post_meta/update_metadata_by_mid/delete_metadata_by_midresponses, since we check for those cases beforehand anyway.

@rachelbaker https://github.com/rachelbaker Can we hit those extra docs
in 1.1 instead? If so, let's #reviewmerge and ship 1.0.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/207#issuecomment-44081981
.

Rachel Baker