[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: |
|
||||||||||||||||
| Issue links: |
|
||||||||||||||||
| 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. All forms in UI inventory needs to be checked. Further details in
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. |