Conversation
|
|
longvo-cv
requested review from
roman-barannyk and
yuntianhu
and removed request for
roman-barannyk
|
could you make correct those errors, please. ERROR: api-schema-lint: ramls/schema/anonymization-result.json: Missing top-level "description". |
|
please make sure the test coverage for your new code is greater than 80% |
yuntianhu
left a comment
yuntianhu
left a comment
There was a problem hiding this comment.
could you mirror loan anon. raml file, please.
There was a problem hiding this comment.
could you add top level description, please. thank you
| import org.folio.circulation.domain.RequestStatus; | ||
| import org.folio.circulation.support.logging.Logging; | ||
|
|
||
|
|
There was a problem hiding this comment.
remove extra space, please
|
|
||
| // Get includeCirculationLogs parameter (default to true) | ||
| boolean includeCirculationLogs = body.getBoolean("includeCirculationLogs", true); | ||
|
|
There was a problem hiding this comment.
remove extra space here, please
yuntianhu
left a comment
yuntianhu
left a comment
There was a problem hiding this comment.
please study this ticket
https://folio-org.atlassian.net/browse/UXPROD-4302?quickFilter=2727&selectedIssue=CIRC-2292
|
3 participants
CIRC-2292: Implement Request Anonymization API
Summary
Adds a POST endpoint to anonymize closed circulation requests in bulk, removing personally identifiable information from request records.
Changes
New Endpoint: POST /circulation/requests/anonymize
Validation: Verifies requests have closed status before anonymization
Delegate to Storage: Calls mod-circulation-storage for actual anonymization
Tests: Unit and integration tests for the new functionality
Schema: Added request/response JSON schemas and RAML definitions