Loan Policy: Add "No due date limit" to "Alternate fixed due date schedule (due date limit) for renewals"

Description

Purpose: Per discussion in the comments of FOLIO-1101, we should also offer the option of no due date limit. Story will be added to add "No due date limit" option and to clarify that, when nothing is selected, the sytem should apply the due date limit from the original loan.

User Story:
As a user who is creating a loan policy
I want to be able to configure an alternate loan period for renewals without any due date limit

Scenarios:

  1. Scenario

    • Given the Loan policy form in Settings > Circulation > Loan policies

    • When displayed

    • Then:

      • The "Alternate fixed due date schedule for renewals" menu should contain the value "No due date limit"

      • The new value should display either at the top or the bottom of the list (not mixed in alphabetically with the fixed due date schedules)

  2. Scenario

    • Given "No due date limit" is selected in the "Alternate fixed due date schedule for renewals" menu

    • When due date is calculated at the time of renewal (loans can be renewed from Users > Loans)

    • Then the due date should not be truncated

Environment

None

Potential Workaround

None

Checklist

hide

TestRail: Results

Activity

Show:

Cate Boerema August 21, 2018 at 8:38 AM

Closing as duplicate of

Jakub Skoczen June 21, 2018 at 12:14 PM

This needs a backend change. We will discuss how this needs to look like in the backend and create a blocker issue (most likley to be scheduled for sprint41)

Marc Johnson June 20, 2018 at 11:50 AM
Edited

The value from the Alternate fixed due date schedule for renewals is typically used to populate the alternateFixedDueDateScheduleId property in the loan policy which refers to a fixed due date schedule (and is currently enforced by a foreign key constraint) and we fetch fixed due date schedules records during loan policy application based upon it.

Given that we fallback to the regular fixed due date schedule if no alternate is present, we cannot simply omit this property in order for none to be applied.

Given the foreign key, it is unlikely we can use a special value to represent the No due date limit choice without changes, so I think we likely need to discuss further how we might represent this in the API.

I wanted to share my thoughts before I forgot them, it might be that the conversation can be deferred given this is currently a P4 issue.

Duplicate

Details

Assignee

Reporter

Labels

Priority

Fix versions

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created June 7, 2018 at 9:04 AM
Updated September 11, 2018 at 6:37 AM
Resolved August 21, 2018 at 8:38 AM
TestRail: Cases
TestRail: Runs