Summary: | ERM - Mandatory fields don't have the 'required' class on the label | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | ERM | Assignee: | Lucas Gass (lukeg) <lucas> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P5 - low | CC: | jonathan.druart, jonathan.field, lucas, martin.renvoize, pedro.amorim |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
23.05.00,22.11.04
|
|
Circulation function: | |||
Bug Depends on: | 32030 | ||
Bug Blocks: | |||
Attachments: |
Bug 32180: Add required class to labels in ERM
Bug 32180: Add required class to labels in ERM Bug 32180: Add required class to labels in ERM |
Description
Jonathan Druart
2022-11-11 15:13:55 UTC
Created attachment 148246 [details] [review] Bug 32180: Add required class to labels in ERM To test: 1. Apply patch, yarn build 2. Add/edit Agreemnets, make sure all required fields have a red 'required' label 3. Check 'Add new period', 'Add new user', 'Add new license', 'Add new related agreement'. 4. Add/edit Licenses, make sure all required fields have a red 'required' label. Created attachment 148253 [details] [review] Bug 32180: Add required class to labels in ERM To test: 1. Apply patch, yarn build 2. Add/edit Agreemnets, make sure all required fields have a red 'required' label 3. Check 'Add new period', 'Add new user', 'Add new license', 'Add new related agreement'. 4. Add/edit Licenses, make sure all required fields have a red 'required' label. Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com> Created attachment 148255 [details] [review] Bug 32180: Add required class to labels in ERM To test: 1. Apply patch, yarn build 2. Add/edit Agreemnets, make sure all required fields have a red 'required' label 3. Check 'Add new period', 'Add new user', 'Add new license', 'Add new related agreement'. 4. Add/edit Licenses, make sure all required fields have a red 'required' label. Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Great work.. Passing QA Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to stable for 22.11.x Not needed for 22.05.x, no backport. |