Remove Inline Editing - Title+Package Show

Priority

Labels

Environment

None

Template

None

Description

Remove Inline Editing from the Title+Package Detail (View Mode)

  • The user may only change the selection status of a package-title (resource) from View Mode

  • All other fields can only be changed when in Full Page Edit mode.

  • When in view mode show the value entered or a statement that no value has been selected/entered

Development Team

None

Release

None

Story Points

Sprint

Attachments

2

Checklist

hide

TestRail: Results

Activity

Show:

Carole GodfreyMay 31, 2018 at 1:18 PM

All fields have been made read only with the exception of custom coverage dates. To be addressed in separate ticket. Ticket should also confirm proper operation of isSelected toggle with other fields

Carole GodfreyMay 29, 2018 at 2:08 PM
Edited

Status (5/27)

Carole GodfreyMay 24, 2018 at 11:30 PM

Added update for coverage statement - https://github.com/folio-org/ui-eholdings/pull/410

Carole GodfreyMay 23, 2018 at 7:11 PM

The following PR - incorporates changes for Visibility https://github.com/folio-org/ui-eholdings/pull/408.
Visibility can only be edited when in full page edit mode. Show page displays text regarding visibility.

Note – Review of this PR noted some issues with clearing out resource values when setting isSelected to false for managed resources. Specifically

model.identifiersList = [];
model.identifiers = {};

Looking at redux layer and mod-kb-ebsco – setting model.identifiers = [] is correct.
Need to review the clearing of resource values in general when setting isSelected to false to be sure we are resetting properly and covering all cases.

Carole GodfreyMay 17, 2018 at 1:23 PM

I am repointing story from 3 to 5 after more research. Want to be sure that all test cases that I am removing for in-line edit are covered in full page edits for custom and or managed resources

Done

Details

Assignee

Reporter

Fix versions

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created May 11, 2018 at 8:15 PM
Updated September 7, 2018 at 1:49 PM
Resolved May 31, 2018 at 1:18 PM
TestRail: Cases
TestRail: Runs