Review Request 112095: add asset/participant/ratings/:assetId

Aaron J. Seigo aseigo at kde.org
Mon Sep 2 13:57:47 UTC 2013



> On Aug. 19, 2013, 8:27 p.m., Aaron J. Seigo wrote:
> > server/routes.js, line 365
> > <http://git.reviewboard.kde.org/r/112095/diff/1/?file=179719#file179719line365>
> >
> >     this should be merged with asset/participant/ratings above as:
> >     
> >     'asset/participant/ratings/:assetId?'
> >     
> >     this makes it consistent with other similar routes that optionally take a asset or other id that narrows the result set down.
> >     
> >     see the 'collection/list/:collectionId?' route for a pattern to follow
> 
> Giorgos Tsiapaliokas wrote:
>     >this should be merged with asset/participant/ratings above as
>     
>     do you mean that 'asset/participant/ratings/:assetId' should be merged with 'participant/ratings'
>     into 'asset/participant/ratings/:assetId?'? I can't find the route 'asset/participant/rating'.

yes, it should be participant/ratings/:assetId?


- Aaron J.


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


On Aug. 15, 2013, 7:11 a.m., Giorgos Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112095/
> -----------------------------------------------------------
> 
> (Updated Aug. 15, 2013, 7:11 a.m.)
> 
> 
> Review request for Bodega.
> 
> 
> Description
> -------
> 
> This patch adds the route asset/participant/ratings/:assetId
> which gives us the user's ratings for a specific asset.
> 
> 
> Diffs
> -----
> 
>   server/doc/bodega.json 3efd1cf 
>   server/lib/bodegadb.js 20a8ef8 
>   server/lib/db/ratings.js e5f4bed 
>   server/routes.js 903c9a3 
>   server/test/ratings.js 8538e86 
> 
> Diff: http://git.reviewboard.kde.org/r/112095/diff/
> 
> 
> Testing
> -------
> 
> the tests are passing
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/active/attachments/20130902/3afc600e/attachment.html>


More information about the Active mailing list