[Koha-bugs] [Bug 29983] Display the pretend claim column in overdue.tt

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Sun Mar 27 12:32:19 CEST 2022


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

Katrin Fischer <katrin.fischer at bsz-bw.de> changed:

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

--- Comment #10 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
Overall this looks great and I am quite happy to see the code has been moved to
an include. Some small 'complaints', hopefully all easy to fix:

1) Translation

I believe we now need to change this to use the single underscore as it was
moved from a .js to an .inc file:

+                content = __("Unable to claim as returned");

2) Column configuration

The new column should be added to the column configuration for this report.
Some libraries are not using the return claims feature and then it's nice when
it can be hidden by default and it also helps when you want to print the table
to slim it down.

3) QA test tools

 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
   FAIL   valid_template
                modal-claims-js: not found

 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt
   FAIL   filters
                missing_filter at line 125 (                <span
class="badge">[% overdueloo.return_claim_created_on %]</span>)

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list