[ 
https://issues.apache.org/jira/browse/SHINDIG-1410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12903381#action_12903381
 ] 

Paul Lindner commented on SHINDIG-1410:
---------------------------------------

looks good so far.

I've picked up a few things along the way:

* use two-space indents
* Add test classes with TODOs for tests, it would be great if you could add 
some tests
* Added default notimplemented implementations.

Could you add more tests by tackling finishing off the stub methods in the test 
classes?  Bonus points for javascript and endtoend tests.


> Albums & MediaItems Implementation
> ----------------------------------
>
>                 Key: SHINDIG-1410
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-1410
>             Project: Shindig
>          Issue Type: New Feature
>          Components: Java
>    Affects Versions: 2.0.0-RC2
>            Reporter: Eric Woods
>            Priority: Minor
>             Fix For: 2.0.0-RC2
>
>         Attachments: 100813 Albums & MediaItems.txt
>
>
> Implementations for the Albums and MediaItems services within Apache Shindig. 
>  Includes the handlers, services, and implementing database as well as 
> relevant bindings. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to