inveniosoftware-attic / invenio-circulation-legacy

https://invenio-circulation.readthedocs.io
GNU General Public License v2.0
4 stars 8 forks source link

global: initial REST validation API #33

Closed mvesper closed 8 years ago

mvesper commented 8 years ago

Signed-off-by: Martin Vesper martin.vesper@cern.ch

jirikuncar commented 8 years ago

What about adding dry-run or only-validate option to the endpoints?

Alternatively, I would change the name from Test* to Validate* and use inheritance instead of specifying the validator twice.

class ValidatorFoo(ValidatorBase, Foo):
    pass
jirikuncar commented 8 years ago

LGTM

mvesper commented 8 years ago

@jirikuncar I saw that you 'thumbed-up' the commit message. I changed it though. Is it still valid? Should be fine otherwise :)