| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | 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. |