[Koha-bugs] [Bug 18399] Reason is missing in suggestion management part

[bugzilla-daemon at bugs.koha-community.org]

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

Nick Clemens <nick at bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nick at bywatersolutions.com
             Status|Signed Off                  |Failed QA
         QA Contact|testopia at bugs.koha-communit |nick at bywatersolutions.com
                   |y.org                       |

--- Comment #10 from Nick Clemens <nick at bywatersolutions.com> ---
Hi Alex,

Why not use Koha::Template::Plugin::AuthorisedValues rather than processing the
category in the pl and passing it?

$(this).next("#other_reason").show();
should be the same as
$("#other_reason").show();

I like the way the 'Other' suggestions work but 'Cancel' is a confusing wording
(if I am editing an existing suggestion I am not 'cancelling' but 'changing') I
would suggest 'Choose an standard value' or something similar

Nice and needed feature.

-Nick

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