Removal of api/swagger/definitions.t

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
Report Content as Inappropriate

Removal of api/swagger/definitions.t

Tomas Cohen Arazi
Hi, I propose removing t/db_dependent/api/swagger/definitions.t for a simple reason: it is not obvious that our endpoints should mimick our object models. For instance, if some fields should not be exposed on the API, then they should not be tested for existence. Also, think of fields that have names that are kept for historical reasons and should better be changed on the published api.

Maybe I went too far explaining it, but I think it should be removed.
Tomás Cohen Arazi
Theke Solutions (https://theke.io)
✆ +54 9351 3513384

Koha-devel mailing list
[hidden email]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/