Re: Review Request 53661: GEODE-2094 Update admin/dev REST API documentation

2016-11-11 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53661/#review155747 --- Ship it! Ship It! - Jinmei Liao On Nov. 11, 2016, 5:37 p.m.,

Re: Review Request 53661: GEODE-2094 Update admin/dev REST API documentation

2016-11-11 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53661/#review155729 --- Ship it! Ship It! - Joey McAllister On Nov. 11, 2016, 5:37 p

Re: Review Request 53661: GEODE-2094 Update admin/dev REST API documentation

2016-11-11 Thread Karen Miller
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53661/ --- (Updated Nov. 11, 2016, 5:37 p.m.) Review request for geode, Dave Barnes, Jinme

Re: Review Request 53661: GEODE-2094 Update admin/dev REST API documentation

2016-11-10 Thread Karen Miller
> On Nov. 10, 2016, 11:48 p.m., Dave Barnes wrote: > > I noticed one variation to the pattern: In > > geode-docs/rest_apps/setup_config.html.md.erb at line 100, did you miss one > > conversion of start-dev-rest-api to start-rest-api, or was this intentional? Good catch. I'll append a fix for

Re: Review Request 53661: GEODE-2094 Update admin/dev REST API documentation

2016-11-10 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53661/#review155668 --- I noticed one variation to the pattern: In geode-docs/rest_apps/s