https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35044

Julian Maurice <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #82 from Julian Maurice <[email protected]> ---
1) In the patches:
> #FIXME: The above $subscription->{additional_fields} may be redundant. Remove 
> if so.
Can it be checked/fixed now ?

2) With the patches it is no longer possible to save a "falsy" value like '' or
'0'. That's a regression and it would be nice to add a test for that.

3) I was a bit surprised when I realized that repeatable AV fields are not
actually repeatable. There is no "+ New" button like with other repeatable
fields.
It makes sense in this case to have a list of checkboxes instead of a
repeatable dropdown list, but I didn't expect that. So maybe the issue is with
the term "repeatable". I think that "multivalued" would fit better.
Is it feasible to change that term without too much work ?
Can anyone else from QA team or release team give their opinion on this ?

Failing QA for 1 and 2

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to