Bug 19255

Summary: Correct explanation about list categories on shelves.pl in staff
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: ListsAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: m.de.rooy, mtompset
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 18980    
Bug Blocks:    
Attachments: Bug 19255: Correct explanation about list types on shelves.pl in staff
Bug 19255: [Follow-up] Typo "will accessible"
Bug 19255: Correct explanation about list types on shelves.pl in staff
Bug 19255: [Follow-up] Typo "will accessible"
Bug 19255: Correct explanation about list types on shelves.pl in staff
Bug 19255: [Follow-up] Typo "will accessible"
Bug 19255: QA Followup - Fix tabs for spaces

Description Marcel de Rooy 2017-09-05 12:45:34 UTC
From Jonathan on bug 18980:
On the staff interface:
"""
    Private list: Is managed by you and can be seen only by you.
    Public list: Can be seen by everybody, but managed only by you.
"""

Should not we update it? If a list is private but shared, the description is wrong.
Comment 1 Marcel de Rooy 2017-09-05 13:49:07 UTC
Created attachment 66853 [details] [review]
Bug 19255: Correct explanation about list types on shelves.pl in staff

The explanation on list categories on the shelves form is removed.
The online help text is actualized.

Test plan:
Go to Lists in staff. Edit a list. The remarks about private and public
lists on the right side are gone.
Click Help. Read the adjusted text. Does it make sense?

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 2 Mark Tompsett 2017-09-06 04:44:17 UTC
I was wondering if you could fix some other grammar while you are at it too.
"Once the list is saved it will accessible"
it will BE accessible.
Comment 3 Marcel de Rooy 2017-09-06 11:02:41 UTC
Created attachment 66868 [details] [review]
Bug 19255: [Follow-up] Typo "will accessible"

This patch speaks for itself.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Mark Tompsett 2017-09-06 16:10:21 UTC
Created attachment 66889 [details] [review]
Bug 19255: Correct explanation about list types on shelves.pl in staff

The explanation on list categories on the shelves form is removed.
The online help text is actualized.

Test plan:
Go to Lists in staff. Edit a list. The remarks about private and public
lists on the right side are gone.
Click Help. Read the adjusted text. Does it make sense?

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 5 Mark Tompsett 2017-09-06 16:10:27 UTC
Created attachment 66890 [details] [review]
Bug 19255: [Follow-up] Typo "will accessible"

This patch speaks for itself.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 6 Mark Tompsett 2017-09-06 16:12:49 UTC
Thanks for the followup addressing comment #2, Marcel.
Comment 7 Nick Clemens (kidclamp) 2017-10-07 00:08:58 UTC
Created attachment 67759 [details] [review]
Bug 19255: Correct explanation about list types on shelves.pl in staff

The explanation on list categories on the shelves form is removed.
The online help text is actualized.

Test plan:
Go to Lists in staff. Edit a list. The remarks about private and public
lists on the right side are gone.
Click Help. Read the adjusted text. Does it make sense?

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 8 Nick Clemens (kidclamp) 2017-10-07 00:09:00 UTC
Created attachment 67760 [details] [review]
Bug 19255: [Follow-up] Typo "will accessible"

This patch speaks for itself.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 9 Nick Clemens (kidclamp) 2017-10-07 00:09:03 UTC
Created attachment 67761 [details] [review]
Bug 19255: QA Followup - Fix tabs for spaces

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 10 Jonathan Druart 2017-10-09 19:19:02 UTC
Pushed to master for 17.11, thanks to everybody involved!