Bug 24790

Summary: POS missing from the 'More' dropdown
Product: Koha Reporter: Tomás Cohen Arazi (tcohen) <tomascohen>
Component: Fines and feesAssignee: Tomás Cohen Arazi (tcohen) <tomascohen>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: 1joynelson, george, martin.renvoize, sally.healey
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00
Circulation function:
Bug Depends on: 23354    
Bug Blocks: 13985    
Attachments: Bug 24790: Add POS to the 'More' dropdown on the staff client
Bug 24790: Add POS to the 'More' dropdown on the staff client
Bug 24790: Add POS to the 'More' dropdown on the staff client
Bug 24790: Add POS to the 'More' dropdown on the staff client

Description Tomás Cohen Arazi (tcohen) 2020-03-03 14:36:09 UTC
We missed to add it.
Comment 1 Tomás Cohen Arazi (tcohen) 2020-03-03 14:46:23 UTC
Created attachment 100034 [details] [review]
Bug 24790: Add POS to the 'More' dropdown on the staff client

This trivial patch adds 'Point of sale' to the dropdown.

To test:
1. Notice there is no 'Point of sale' entry in the staff dropdown
=> FAIL: No dropdown
2. Apply this patch and reload
=> SUCCESS: Point of sale displays and works
3. Play with enabling/disabling the feature and removing permissions
=> SUCCESS: The entry in the dropdown respects the configured
permissions and settings.
4. Sign off :-D
Comment 2 ByWater Sandboxes 2020-03-03 15:14:29 UTC
Created attachment 100038 [details] [review]
Bug 24790: Add POS to the 'More' dropdown on the staff client

This trivial patch adds 'Point of sale' to the dropdown.

To test:
1. Notice there is no 'Point of sale' entry in the staff dropdown
=> FAIL: No dropdown
2. Apply this patch and reload
=> SUCCESS: Point of sale displays and works
3. Play with enabling/disabling the feature and removing permissions
=> SUCCESS: The entry in the dropdown respects the configured
permissions and settings.
4. Sign off :-D

Signed-off-by: George Williams <george@nekls.org>
Comment 3 PTFS Europe Sandboxes 2020-03-03 15:27:46 UTC
Created attachment 100040 [details] [review]
Bug 24790: Add POS to the 'More' dropdown on the staff client

This trivial patch adds 'Point of sale' to the dropdown.

To test:
1. Notice there is no 'Point of sale' entry in the staff dropdown
=> FAIL: No dropdown
2. Apply this patch and reload
=> SUCCESS: Point of sale displays and works
3. Play with enabling/disabling the feature and removing permissions
=> SUCCESS: The entry in the dropdown respects the configured
permissions and settings.
4. Sign off :-D

Signed-off-by: George Williams <george@nekls.org>

Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Comment 4 Sally 2020-03-04 10:49:14 UTC
*** Bug 24791 has been marked as a duplicate of this bug. ***
Comment 5 Martin Renvoize (ashimema) 2020-03-04 16:17:50 UTC
Created attachment 100140 [details] [review]
Bug 24790: Add POS to the 'More' dropdown on the staff client

This trivial patch adds 'Point of sale' to the dropdown.

To test:
1. Notice there is no 'Point of sale' entry in the staff dropdown
=> FAIL: No dropdown
2. Apply this patch and reload
=> SUCCESS: Point of sale displays and works
3. Play with enabling/disabling the feature and removing permissions
=> SUCCESS: The entry in the dropdown respects the configured
permissions and settings.
4. Sign off :-D

Signed-off-by: George Williams <george@nekls.org>
Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Martin Renvoize (ashimema) 2020-03-04 16:19:10 UTC
Wow, super quick signoff guys, thanks to all for following up and getting this added before I had a chance to do it and then great teamwork SO..

Passing QA and Pushing.
Comment 7 Martin Renvoize (ashimema) 2020-03-04 16:19:28 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 8 Joy Nelson 2020-04-01 21:19:28 UTC
not pushed to 19.11.x due to dependencies