Uploaded image for project: 'mod-circulation'
  1. mod-circulation
  2. CIRC-1091

Remove override-renewal-by-barcode endpoint

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Closed (View Workflow)
    • Priority: TBD
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 22.0.0
    • Labels:
      None
    • Template:
    • Sprint:
      EPAM-Veg Sprint 111, EPAM-Veg Sprint 112, EPAM-Veg Sprint 113, EPAM-Veg Sprint 114, EPAM-Veg Sprint 115
    • Story Points:
      8
    • Development Team:
      Vega
    • Back End Estimate:
      Large < 10 days
    • Confidence factor:
      Medium

      Description

      This is a spillover (technical debt) from R1 "block override" features.

      Currently, only patron block can be overridden with a regular renew-by-barcode endpoint. If there are any other blocks, they need to be overridden by a call to override-renewal-by-barcode (patron block is also overridden in this case).

      All of these blocks need to become a single block that can be overridden with a call to a regular renew-by-barcode endpoint, just like patron block. Functionality of the override-renewal-by-barcode endpoint should be transferred to renew-by-barcode. After that, override-renewal-by-barcode should be removed and all of it's tests should be rewritten to use the new API.

      List of errors that are overridden by override-renewal-by-barcode (taken from FE code) that should be combined to a single block:

      • loan is not renewable
      • item is not loanable
      • item is Declared lost
      • item is Aged to lost
      • renewal date falls outside of date ranges in fixed loan policy
      • items cannot be renewed when there is an active recall request
      • loan at maximum renewal number
      • renewal date falls outside of date ranges in the loan policy
      • renewal would not change the due date
      • profile "" in the loan policy is not recognised

        TestRail: Results

          Attachments

            Issue Links

              Activity

                People

                Assignee:
                roman-barannyk Roman Barannyk
                Reporter:
                oleksandrkurash Alexander Kurash
                Back End Estimator:
                Alexander Kurash Alexander Kurash
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    TestRail: Runs

                      TestRail: Cases