Add validation for claimingActive, claimingInterval fields

CSP Request Details

None

CSP Rejection Details

None

CSP Approved

None

Description

Overview: Different APIs can create POL and TItles and we should add validation that claminingInterval is bigger than 0 if claimingActive is true

Acceptance criteria:

  • POST and PUT methods of Purchase Order API are updated to add validation that claminingInterval is bigger than 0 if claimingActive set to true

  • POST and PUT methods of Titles API are updated to add validation that claminingInterval is bigger than 0 if claimingActive set to true

Environment

None

Potential Workaround

None

Attachments

1

Checklist

hide

TestRail: Results

Activity

Show:

Serhii_Nosko December 14, 2023 at 7:45 AM

Tested on https://folio-snapshot.dev.folio.org/ , looks good, BE provides necessary error codes when claimingActive but calimingInterval is not set or less than 1. This validation will be useful for API usage, for UI we have story to prevent saving po line/title with invalid claiming config:  

Create POL with invalid claiming config:

Edit POL with invalid claiming config:

Edit Title with invalid claiming config:

Closing this story

Done

Details

Assignee

Reporter

Tester Assignee

Priority

Story Points

Sprint

Development Team

Thunderjet

Fix versions

Release

Quesnelia (R1 2024)

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created November 28, 2023 at 11:56 AM
Updated December 14, 2023 at 7:45 AM
Resolved December 14, 2023 at 7:45 AM
TestRail: Cases
TestRail: Runs