Review Request 110752: Bodega-server: Add documentation support to bodega-server. Also add markdown support

Zack Rusin zack at kde.org
Sun Jun 2 17:39:06 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110752/#review33590
-----------------------------------------------------------



server/test/support/http.js
<http://git.reviewboard.kde.org/r/110752/#comment24818>

    No. Please fix this.
    
    It's really irritating to see ten pages of text to show up when running make test.
    Either add another argument to getUrl to specify that the output isn't json or use something else than getUrl to test non-json paths.


- Zack Rusin


On June 2, 2013, 3:29 p.m., Antonis Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110752/
> -----------------------------------------------------------
> 
> (Updated June 2, 2013, 3:29 p.m.)
> 
> 
> Review request for Bodega.
> 
> 
> Description
> -------
> 
> Hello
> 
> with that patch i am adding the following support to the bodega-server:
> 
> 1) server/doc is the place which all of our documentaion lives.
> 2) The docs are accessible from /api/filename.E.x. for docs/README.markdown => api/README
> 
> If you all agree with that patch, i will push it to the branch which i am referring above(kokeroulis/markdown/documentation)
> 
> P.S. the two files are from bodega-server and bodega-client
> 
> 
> Diffs
> -----
> 
>   server/doc/index.markdown PRE-CREATION 
>   server/doc/json_api.markdown PRE-CREATION 
>   server/package.json 09f2527 
>   server/routes.js 95883a9 
>   server/test/api.js PRE-CREATION 
>   server/test/support/http.js b9e5cb3 
> 
> Diff: http://git.reviewboard.kde.org/r/110752/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Antonis Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/active/attachments/20130602/27c4a6aa/attachment-0001.html>


More information about the Active mailing list