We don't yet expose the lists functionality over API's, this would be a beneficial enhancement both for internal use and thrid party integrations wishing to integrate with this functionality.
Created attachment 174807 [details] [review] Bug 38050: create admin CRUD endpoint definitions this patch adds api endpoints for lists/virtualshelves. Tests to follow
Created attachment 174808 [details] [review] Bug 38050: create methods for admin CRUD list endpoints This patch implements the methods for the admin CRUD endpoints for lists/virtualshelves
Created attachment 174809 [details] [review] Bug 38050: create public CRUD list endpoint definitions This patch creates the yaml definitions for the public CRUD enpoints for lists/virtualshelves
Created attachment 174810 [details] [review] Bug 38050: create methods for public CRUD list endpoints This patch creates the methods for the public CRUD enpoints for lists/virtualshelves
You're going to need to add unit tests for all these for QA Jacob.
Created attachment 176462 [details] [review] Bug 38050: create admin CRUD endpoint definitions This patch creates the yaml definitions for the CRUD enpoints for virtualshelves/lists
Created attachment 176463 [details] [review] Bug 38050: create methods for admin CRUD list endpoints This patch implements the methods for the admin CRUD endpoints for lists/virtualshelves
Created attachment 176464 [details] [review] Bug 38050: create public CRUD list endpoint definitions This patch creates the yaml definitions for the public CRUD enpoints for lists/virtualshelves
Created attachment 176465 [details] [review] Bug 38050: create methods for public CRUD list endpoints This patch creates the methods for the public CRUD enpoints for lists/virtualshelves
Created attachment 176466 [details] [review] Bug 38050: create tests for admin CRUD list endpoints
Created attachment 176467 [details] [review] Bug 38050: create tests for public CRUD list endpoints
This patch adds API endpoints for lists, both public and admin. To Test: prove /t/db_dependent/api/v1/lists.t prove /t/db_dependent/api/v1/public/lists.t These currently respect "allow_changes_from_others" and "allow_changes_from_staff" as to whether you can modify a given list with the admin endpoints. The public endpoints can only modify lists owned by the authenticated user. As a consequence, to allow you to be able to always edit any lists you create with the admin endpoints (even with another user as the owner) unless changed, any created lists will be created with "allow_changes_from_staff=1" Additionally, passing ->filter_by_readable also means that private lists are not displayed, even with the admin endpoints. Is this intended functionality here or should the admin endpoints (which require administer lists permissions) allow you to edit/view all lists regardless?
Created attachment 176769 [details] [review] Bug 38050: create admin CRUD endpoint definitions This patch creates the yaml definitions for the CRUD enpoints for virtualshelves/lists Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 176770 [details] [review] Bug 38050: create methods for admin CRUD list endpoints This patch implements the methods for the admin CRUD endpoints for lists/virtualshelves Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 176771 [details] [review] Bug 38050: create public CRUD list endpoint definitions This patch creates the yaml definitions for the public CRUD enpoints for lists/virtualshelves Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 176772 [details] [review] Bug 38050: create methods for public CRUD list endpoints This patch creates the methods for the public CRUD enpoints for lists/virtualshelves Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 176773 [details] [review] Bug 38050: create tests for admin CRUD list endpoints Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 176774 [details] [review] Bug 38050: create tests for public CRUD list endpoints Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>