Attachment 147290 Details for Bug 32495 – Bug 32495: Mis-matched required fields in UI and api

[patch] Bug 32495: Mis-matched required fields in UI and api

Bug-32495-Mis-matched-required-fields-in-UI-and-ap.patch (text/plain), 1.73 KB, created by Marcel de Rooy on 2023-02-24 10:46:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-02-24 10:46:59 UTC

Size: 1.73 KB

patch

obsolete

>From 31bbc0288b0acdfcaa25e5f884aed6ae5b5f39fe Mon Sep 17 00:00:00 2001
>From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
>Date: Mon, 9 Jan 2023 15:12:18 +0000
>Subject: [PATCH] Bug 32495: Mis-matched required fields in UI and api
>Content-Type: text/plain; charset=utf-8
>
>The "description" field is marked as required in the UI when creating a new Agreement. This flag has been removed to match the API where it is not a required field. Is Perpetual has not been marked as required as it is a checkbox with a default selection so will always provide a value.
>
>Test plan:
>1) In the ERM module, add a new agreement and observe that the Description field is required
>2) Apply patch
>3) Repeat step 1 but this time the Required flag will not be visible.
>
>Signed-off-by: David Nind <david@davidnind.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> .../prog/js/vue/components/ERM/AgreementsFormAdd.vue           | 3 ---
> 1 file changed, 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue
>index bcfd66111d..2872d9f3f9 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue
>+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue
>@@ -48,9 +48,6 @@
>                                     cols="50"
>                                     required
>                                 />
>-                                <span class="required">{{
>-                                    $__("Required")
>-                                }}</span>
>                             </li>
>                             <li>
>                                 <label for="agreement_status"
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 32495: 145149 | 145205 | 147290 | 147555 | 147558