Bug 39900 - Add public REST endpoint for additional_contents
Summary: Add public REST endpoint for additional_contents
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: REST API (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Pedro Amorim
QA Contact: Tomás Cohen Arazi (tcohen)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-05-14 12:32 UTC by Pedro Amorim
Modified: 2025-06-03 14:27 UTC (History)
5 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 39900: Preparation: Refactor public_query_search_params for reusability (3.00 KB, patch)
2025-05-14 12:41 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 39900: Preparation: Move html customizations options to class for reusability (4.61 KB, patch)
2025-05-14 12:41 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 39900: REST API specs and class changes (8.56 KB, patch)
2025-05-14 12:41 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 39900: Add translated_contents embed (2.36 KB, patch)
2025-05-14 12:41 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 39900: Add tests (3.72 KB, patch)
2025-05-14 12:41 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 39900: Preparation: Refactor public_query_search_params for reusability (3.04 KB, patch)
2025-06-01 20:07 UTC, David Nind
Details | Diff | Splinter Review
Bug 39900: Preparation: Move html customizations options to class for reusability (4.65 KB, patch)
2025-06-01 20:07 UTC, David Nind
Details | Diff | Splinter Review
Bug 39900: REST API specs and class changes (8.60 KB, patch)
2025-06-01 20:07 UTC, David Nind
Details | Diff | Splinter Review
Bug 39900: Add translated_contents embed (2.40 KB, patch)
2025-06-01 20:08 UTC, David Nind
Details | Diff | Splinter Review
Bug 39900: Add tests (3.76 KB, patch)
2025-06-01 20:08 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Pedro Amorim 2025-05-14 12:32:56 UTC

    
Comment 1 Pedro Amorim 2025-05-14 12:41:40 UTC Comment hidden (obsolete)
Comment 2 Pedro Amorim 2025-05-14 12:41:43 UTC Comment hidden (obsolete)
Comment 3 Pedro Amorim 2025-05-14 12:41:46 UTC Comment hidden (obsolete)
Comment 4 Pedro Amorim 2025-05-14 12:41:48 UTC Comment hidden (obsolete)
Comment 5 Pedro Amorim 2025-05-14 12:41:51 UTC Comment hidden (obsolete)
Comment 6 David Nind 2025-06-01 20:07:49 UTC
Created attachment 182887 [details] [review]
Bug 39900: Preparation: Refactor public_query_search_params for reusability

Ensure no regressions are introduced, run:
prove t/db_dependent/Koha/AdditionalContents.t

Signed-off-by: David Nind <david@davidnind.com>
Comment 7 David Nind 2025-06-01 20:07:52 UTC
Created attachment 182888 [details] [review]
Bug 39900: Preparation: Move html customizations options to class for reusability

Signed-off-by: David Nind <david@davidnind.com>
Comment 8 David Nind 2025-06-01 20:07:55 UTC
Created attachment 182889 [details] [review]
Bug 39900: REST API specs and class changes

Signed-off-by: David Nind <david@davidnind.com>
Comment 9 David Nind 2025-06-01 20:08:01 UTC
Created attachment 182890 [details] [review]
Bug 39900: Add translated_contents embed

Signed-off-by: David Nind <david@davidnind.com>
Comment 10 David Nind 2025-06-01 20:08:16 UTC
Created attachment 182891 [details] [review]
Bug 39900: Add tests

prove t/db_dependent/api/v1/additional_contents.t

Signed-off-by: David Nind <david@davidnind.com>
Comment 11 David Nind 2025-06-01 20:11:29 UTC
I tested by running the tests. If this is insufficient, please change the status back to needs signoff.

Testing notes (using KTD):
1. Apply the patches.
2. Rebuild everything (not sure that this step is required): yarn build
3. Ran the tests and these passed:
   - prove t/db_dependent/Koha/AdditionalContents.t
   - prove t/db_dependent/api/v1/additional_contents.t