Skip to content

feat(providers): Add access types to eholdings/providers/{id}/packages response - #710

Merged
viacheslavkol merged 3 commits into
masterfrom
MODKBEKBJ-839
Sep 2, 2026
Merged

feat(providers): Add access types to eholdings/providers/{id}/packages response#710
viacheslavkol merged 3 commits into
masterfrom
MODKBEKBJ-839

Conversation

@viacheslavkol

@viacheslavkol viacheslavkol commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Purpose

Add access types to eholdings/providers/{id}/packages response

Approach

  • Add field to response
  • Add optional includes API parameter to include new field in response

Changes Checklist

  • API Changes: Document any API paths, methods, request or response bodies changed, added, or removed.
  • Database Schema Changes: Indicate any database schema changes and their impact. Confirm that migration scripts were created.
  • Interface Version Changes: Indicate any changes to interface versions.
  • Interface Dependencies: Document added or removed dependencies.
  • Permissions: Document any changes to permissions.
  • Logging: Confirm that logging is appropriately handled.
  • Unit Testing: Confirm that changed classes were covered by unit tests.
  • Integration Testing: Confirm that changed logic was covered by integration tests.
  • Manual Testing: Confirm that changes were tested on local or dev environment.
  • NEWS: Confirm that the NEWS file is updated with relevant information about the changes made in this pull request.

Related Issues

MODKBEKBJ-839

@viacheslavkol
feat(providers): Add access types to eholdings/providers/{id}/package…
505c465 
…s response

- Add field to response
- Add optional includes API parameter to include new field in response

Implements: MODKBEKBJ-839
@viacheslavkol viacheslavkol self-assigned this Aug 28, 2026
@viacheslavkol
viacheslavkol marked this pull request as ready for review August 31, 2026 09:43
@viacheslavkol
viacheslavkol requested a review from a team as a code owner August 31, 2026 09:43
psmagin
psmagin reviewed Aug 31, 2026
View reviewed changes
Comment thread src/main/java/org/folio/rest/impl/EholdingsProvidersImpl.java
@sonarqubecloud

sonarqubecloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@viacheslavkol
viacheslavkol merged commit a1c3c42 into master Sep 2, 2026
18 checks passed
@viacheslavkol
viacheslavkol deleted the MODKBEKBJ-839 branch September 2, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@psmagin psmagin psmagin approved these changes
@Aliaksandr-Fedasiuk Aliaksandr-Fedasiuk Aliaksandr-Fedasiuk approved these changes

Assignees

@viacheslavkol viacheslavkol

Labels

ai

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@viacheslavkol @psmagin @Aliaksandr-Fedasiuk