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

Katrin Fischer <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Passed QA                   |Failed QA

--- Comment #19 from Katrin Fischer <[email protected]> ---
Some small stuff, please follow-up:

1) Database update

It's not using say_success, but it's also not giving us any success message :)

2) What is actually meant by "manage"? 

I assume that "manage" actually means "edit". While any of the permissions
allow to "view".

If that is the case, I feel it would be good to adjust the permission
descriptions accordingly.

There is a slight discrepancy tho:
With only create permission, I have the entry to suggestions in the "more"
menu, but it's missing from the navigation in the acquisition module.
I can also not see the pending suggestions on the start page.

I think we should adjust the navigation in the acq module at least.

Should the pending suggestions on acq and staff interface home page be visible
with manage_suggestions only or also with any suggestion permission? 

3) Question

+        <div class="note">Only staff with superlibrarian or full suggestions
permissions are returned in the search results</div>

I think this is in line with the previous behavior, but I wonder if we should
not change to allow "any" instead of "full" here. 

I assume this is for setting the manager of a suggestion - editing might be
required, but not sure if delete and create would be as well.

-- 
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