Modify default values and validation for 'Lost item fee policy'

Description

Please see https://docs.google.com/spreadsheets/d/1rX4vuPZXBm-fBBAWWtqQ67HGDABywp_b8FLsFuYrrPM/edit?usp=sharing for details related to this user story:

  • Worksheet New defaults shows the six default changes.

  • Worksheet New validation lists the validations required for each field (which replaces any existing validations).

Environment

None

Potential Workaround

None

Attachments

1

Checklist

hide

TestRail: Results

Activity

Show:

Holly Mistlebauer October 13, 2020 at 7:10 PM

and : I just tested this on SNAPSHOT DEV and found some validation errors. I have reported them in a single bug ticket ().

Maxim Didenko October 13, 2020 at 6:51 AM

I reviewed 5 cases described above and all works as expected:
1. New requirements covered in scope of https://folio-org.atlassian.net/browse/UICIRC-503
2. Correct message
3. Message was updated by Holly in Slack but did not reflected in user story
4. Message was updated by Holly in Slack but did not reflected in user story
5. Correct message


cc: ,

Darcy Branchini October 7, 2020 at 8:33 PM

, please see the comment above for feedback after my review. cc:

Darcy Branchini October 7, 2020 at 7:42 PM
Edited

1. If 'Set cost' selected and = 0 plus 'Lost item processing fee' = 0 ask user to confirm:

Confirm that user wants to save policy with no lost fee charges set (this is a new requirement is covered by user story )

  • , I can't get the page to allow me to do this. It seems like it's a contradiction to other validations?

2. Lost item processing fee > 0, then 'Charge lost item processing fee if item declared lost by patron ?' or 'Charge lost item processing fee if item aged to lost by system ?' must be 'Yes'

Under 'Lost item processing fee': 'Lost item processing fee' cannot be > 0 unless 'Charge lost item processing fee if item declared lost by patron ?' and/or 'Charge lost item processing fee if item aged to lost by system ?' is 'Yes'

  • Incorrect validation message (correct message above).

3. If lost item replaced, remove lost item processing fee is 'Yes', 'Replacement allowed' must be 'Yes'

Under 'If lost item replaced, remove lost item processing fee': Cannot be 'Yes' unless 'Replacement allowed' also 'Yes'

  • Incorrect validation message (correct message above) and error in validation. I did select Yes for Replacement allowed and I still get a validation error for this field.

4. If lost item replaced, remove lost item processing fee is 'Yes', 'Lost item processing fee' must be > 0.00

Under 'If lost item replaced, remove lost item processing fee': 'Lost item processing fee' must be > 0 for this selection

  • Couldn't test this because of the validation error listed in #3.

5. If replacement processing fee > 0.00, 'Replacement allowed' must be 'Yes'

Under 'Replacement processing fee': 'Replacement processing fee' cannot be > 0 unless 'Replacement allowed' is 'Yes'

  • Incorrect validation message (correct message above).

Done

Details

Assignee

Reporter

Tester Assignee

Priority

Story Points

Sprint

Development Team

Vega

Fix versions

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created September 14, 2020 at 2:40 PM
Updated October 13, 2020 at 7:10 PM
Resolved October 13, 2020 at 1:03 PM
TestRail: Cases
TestRail: Runs