Bug 22138 - members/paycollect.pl has not been updated to have the new tab names
Summary: members/paycollect.pl has not been updated to have the new tab names
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Fines and fees (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low minor
Assignee: Jasmine Amohia
QA Contact: Testopia
URL:
Keywords: Academy
Depends on:
Blocks:
 
Reported: 2019-01-16 00:37 UTC by Aleisha Amohia
Modified: 2019-10-14 19:59 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 22138: updated tab names on members/paycollect.pl (1.59 KB, patch)
2019-01-16 01:07 UTC, Jasmine Amohia 		Details | Diff | Splinter Review
Bug 22138: updated tab names on members/paycollect.pl (1.64 KB, patch)
2019-01-16 09:02 UTC, ByWater Sandboxes 		Details | Diff | Splinter Review
Bug 22138: updated tab names on members/paycollect.pl (1.69 KB, patch)
2019-01-16 20:07 UTC, Katrin Fischer 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Aleisha Amohia 2019-01-16 00:37:39 UTC 
Under the new Accounting tab (was "Fines") the tabs have also been renamed - 'Account' became 'Transactions', and 'Pay fines' became 'Make a payment'.

These tab headings need to be updated on members/paycollect.pl
Comment 1 Jasmine Amohia 2019-01-16 01:07:57 UTC 
Created attachment 84033 [details] [review]
Bug 22138: updated tab names on members/paycollect.pl

Test plan:
1. Give patron outstanding fees
2. Search patron and select 'Make payment'
3. Select either 'Pay' or 'Write off'
4. Confirm that first tab title remains as 'Transactions'
5. Confirm that current tab title remains as 'Make a payment'
Comment 2 ByWater Sandboxes 2019-01-16 09:02:40 UTC 
Created attachment 84045 [details] [review]
Bug 22138: updated tab names on members/paycollect.pl

Test plan:
1. Give patron outstanding fees
2. Search patron and select 'Make payment'
3. Select either 'Pay' or 'Write off'
4. Confirm that first tab title remains as 'Transactions'
5. Confirm that current tab title remains as 'Make a payment'

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2019-01-16 09:25:20 UTC 
Nice work Jasmine!

This works as intended, and I've signed it off.

For future test plans it might help if they are a little bit more detailed (this makes it easier and quicker for sign off and QA), for example:

1. Search for a patron (Patrons > Search > Select a patron)
2. Add an invoice for a patron (Accounting > Create manual invoice)
3. Before applying the patch check that when you click on the Pay and Write off buttons (under Accounting > Make a payment) the tab titles change to the old terms used 'Account' and 'Pay fines'
4. After applying the patch click on the Pay and Write off buttons again
5. Confirm that first tab title is now 'Transactions'
6. Confirm that second tab title is now 'Make a payment'

I was able to figure it out, so no worries.

Keep up the good work!

David
Comment 4 Katrin Fischer 2019-01-16 20:07:19 UTC 
Created attachment 84080 [details] [review]
Bug 22138: updated tab names on members/paycollect.pl

Test plan:
1. Give patron outstanding fees
2. Search patron and select 'Make payment'
3. Select either 'Pay' or 'Write off'
4. Confirm that first tab title remains as 'Transactions'
5. Confirm that current tab title remains as 'Make a payment'

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Katrin Fischer 2019-01-16 20:08:12 UTC 
Good catch!
Comment 6 Nick Clemens (kidclamp) 2019-01-28 14:16:51 UTC 
Awesome work all!

Pushed to master for 19.05
Comment 7 Martin Renvoize (ashimema) 2019-02-01 12:42:56 UTC 
Pushed to 18.11.x for 18.11.03
Comment 8 Jesse Maseto 2019-02-06 17:40:16 UTC 
Merge conflict with 18.05.x. Please rebase if needed in 18.05.x.