Bug 35219

Summary: ERM usage endpoints not showing up in documentation
Product: Koha Reporter: Tomás Cohen Arazi (tcohen) <tomascohen>
Component: REST APIAssignee: Matt Blenkinsop <matt.blenkinsop>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: blocker    
Priority: P5 - low CC: fridolin.somers, matt.blenkinsop, tomascohen
Version: unspecifiedKeywords: rel_23_11_candidate
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00
Circulation function:
Bug Depends on: 34587    
Bug Blocks:    
Attachments: Bug 35219: Fix ERM usage endpoints to show in docs
Bug 35219: Fix ERM usage endpoints to show in docs
Bug 35219: Fix ERM usage endpoints to show in docs

Description Tomás Cohen Arazi (tcohen) 2023-11-01 18:02:55 UTC
Due to the lack of pointers from swagger.yaml to the tags used on the feature's endpoints, there's no mention to them in the docs [1]

Before doing it, please beware the tags should all be scoped with the `erm_` prefix to avoid future problems.


[1] https://api.koha-community.org/Development.html
Comment 1 Matt Blenkinsop 2023-11-02 14:04:20 UTC
Created attachment 158263 [details] [review]
Bug 35219: Fix ERM usage endpoints to show in docs

This patch fixes the tags and also adds the tags to the swagger.yaml file to allow the endpoints to be documented correctly.

One endpoint has also been deleted as it is no longer required.

Test plan:
Check the attached files to see that all tags are now prefixed with 'erm_' and that the swagger file now includes an entry for all of these files
Comment 2 Owen Leonard 2023-11-03 12:34:05 UTC
Created attachment 158333 [details] [review]
Bug 35219: Fix ERM usage endpoints to show in docs

This patch fixes the tags and also adds the tags to the swagger.yaml
file to allow the endpoints to be documented correctly.

One endpoint has also been deleted as it is no longer required.

Test plan:
Check the attached files to see that all tags are now prefixed with
'erm_' and that the swagger file now includes an entry for all of these
files

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Katrin Fischer 2023-11-04 18:15:03 UTC
Created attachment 158395 [details] [review]
Bug 35219: Fix ERM usage endpoints to show in docs

This patch fixes the tags and also adds the tags to the swagger.yaml
file to allow the endpoints to be documented correctly.

One endpoint has also been deleted as it is no longer required.

Test plan:
Check the attached files to see that all tags are now prefixed with
'erm_' and that the swagger file now includes an entry for all of these
files

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Tomás Cohen Arazi (tcohen) 2023-11-06 11:40:55 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 5 Fridolin Somers 2023-11-10 07:03:11 UTC
Depends on Bug 34587 not in 23.05.x