[UIIN-490] UI-Inventory. Required field display handling Update (Instance, Holdings, Item) Created: 18/Feb/19  Updated: 17/May/19  Resolved: 17/May/19

Status: Closed
Project: ui-inventory
Components: None
Affects versions: None
Fix versions: 1.8.0

Type: Bug Priority: P2
Reporter: Charlotte Whitt Assignee: Zak Burke
Resolution: Done Votes: 0
Labels: bug-layout, front-end, q1-2019-ux-consistency, triaged, ui-only, ux-clone-story
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original estimate: Not Specified

Attachments: PNG File Skärmavbild 2019-02-20 kl. 21.57.57.png     PNG File Skærmbillede 2019-02-15 kl. 19.22.51.png    
Issue links:
Cloners
clones FOLIO-1784 Clone User Story: Required field disp... Closed
Relates
relates to UIIN-493 UI-Inventory. Double asterisks for re... Closed
Sprint: Core: F - Sprint 60, Core: F - Sprint 61, Core: F - Sprint 62, Core: F - Sprint 63
Story Points: 5
Development Team: Prokopovych
Tester Assignee: Charlotte Whitt

 Description   

Overview: Stripes has changed the rendering of mandatory versus non mandatory fields. And UI-Inventory needs to align. See for example non mandatory fields are marked as mandatory, which are not mandatory.


In this example - then the only required element is: Item HRID.

All forms in UI inventory needs to be checked.

Further details in FOLIO-1784 Closed .

The front-end change is trivial assuming this is just a labeling mistake and there is no actual discrepancy with required fields on the back-end.

The elements which are required for instance, holdings, and item records can be verified in: Required Elements Inventory Beta - Metadata Elements



 Comments   
Comment by Ann-Marie Breaux (Inactive) [ 21/Feb/19 ]

Just a quick update - looks like this bug is blocking users from manually creating item records on folio-testing and folio-snapshot

Comment by Charlotte Whitt [ 25/Mar/19 ]

Ann-Marie Breaux, no this is a `ui-only` thing, so this is not blocking users from doing manual creation of item records.

Comment by Cate Boerema (Inactive) [ 29/Mar/19 ]

Charlotte Whitt this bug will be difficult to test as written. Can you please include a list of required fields for each of the record types?

Comment by Charlotte Whitt [ 29/Mar/19 ]

Cate Boerema and Zak Burke: All required elements are listed in my spreadsheet: Required Elements Inventory Beta - Metadata Elements - https://docs.google.com/spreadsheets/d/1tTxIdCX-tzzHSKRhAP3iIXHpGyNmDTstV_DbB4P8HN8/edit#gid=952741439

Comment by Cate Boerema (Inactive) [ 10/May/19 ]

Zak Burke it looks like this issue has been in code review since 09/Apr/19 . Is it ready for manual testing now?

Comment by Zak Burke [ 10/May/19 ]

Cate Boerema, it's finally ready; just moved it. This ticket was 10% UI changes and 90% writing new unit tests to placate the automatic PR code review system. I forgot to move it to "In review" once I finally completed the tests.

Comment by Charlotte Whitt [ 17/May/19 ]

Manual test in FOLIO Snapshot (https://folio-snapshot.aws.indexdata.com/), version @folio/inventory 1.8.1000478, using Chrome.

All looks good. And I'll close the ticket.

Generated at Thu Feb 08 23:16:04 UTC 2024 using Jira 1001.0.0-SNAPSHOT#100246-sha1:7a5c50119eb0633d306e14180817ddef5e80c75d.