[Koha-bugs] [Bug 17698] Make patron notes show up on staff dashboard

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

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

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

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

--- Comment #57 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
1. Bad resolution conflict, permission self_checkout is re-add (I will fix)

2. The IGNORE modifier is missing in the INSERT statement (I will fix)

3. In the JS console: "ReferenceError: $ is not defined", I did not investigate
it.

4. When I hit /circ/checkout-notes.pl from the side bar, the page displays
"Checkout notes", nothing else. We should add "There is no checkout notes". (I
will fix)

5. The alert id=error is displayed then hide in JS, but it's then displayed
half a second. We should hide it by default (css)

6. I would move the "mark seen" and "mark not seen" buttons at the top of the
table

7. I would display the table on the confirmation screen as well (I will fix)

8. Cursor on "Select all" and "Clear all" links must be adjusted

9. $(".btn-xs").click(function(event){
The selector should be $("button.seen, button.notseen"), you do not want to
apply this function to all other btn-xs on the page (maybe there are only two
for now, but who knows later?)

10. html filters are missing (I will fix)

11. span element should surround translatable string, to help translators (I
will fix).

12. Important: When a note is updated, it's still marked as seen. Is it the
expected behavior?

13. What will happen when hundred of notes will be on this table? Not blocker
but we will need a "hide seen" buttons to filters the already seen notes.

14. patron-title.inc must be used to display patron's info (I will fix)

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