Bug 23212 - Patron request modifications enhancement
Summary: Patron request modifications enhancement
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-25 22:52 UTC by Christopher Brannon
Modified: 2020-10-26 15:02 UTC (History)
4 users (show)

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


Attachments
Bug 25670: Unit tests (5.04 KB, patch)
2020-10-26 10:19 UTC, Séverine Queune
Details | Diff | Splinter Review
Bug 25670: Add filter_by_active and filter_by_id_including_transfers (2.90 KB, patch)
2020-10-26 10:19 UTC, Séverine Queune
Details | Diff | Splinter Review
Bug 20212: Add more embeddable objects to orders (11.49 KB, patch)
2020-10-26 10:19 UTC, Séverine Queune
Details | Diff | Splinter Review
Bug 20212: Clean parcel.pl (5.76 KB, patch)
2020-10-26 10:19 UTC, Séverine Queune
Details | Diff | Splinter Review
Bug 20212: Add missing relation in Biblio (1.12 KB, patch)
2020-10-26 10:19 UTC, Séverine Queune
Details | Diff | Splinter Review
Bug 20212: Use API to render orders datatable (38.55 KB, patch)
2020-10-26 10:19 UTC, Séverine Queune
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Christopher Brannon 2019-06-25 22:52:24 UTC
Currently patron request modifications are an all or none request.  Sometimes requests are entered with mistakes.  Sometimes patrons are requesting some changes that are questionable, like a complete name change, or removing a birth date.

I would like to suggest that each line of request have an approve, deny, or follow-up radio box.  If follow-up is chosen, staff have the ability to add a question, and the question is posed to the patron when they log in to the OPAC.  Maybe an e-mail notification of the question, if they have e-mail.  Once they answer that question, that particular line request is submitted again with the staff question and patron answer.

I would also like to suggest that the new value be editable, so if staff need to format something, or fix an obvious mistake, they can.  Right now, if we see something like this, we have to click on the patron link to open the account, accept the changes, refresh the patron page, and fix the mistake.
Comment 1 Katrin Fischer 2019-07-30 21:39:34 UTC
Hi Christopher, 
I like all of your ideas :) Just wondering if splitting this into several would be nicer (and advertise it better than the general bug title). It could be:

- Select which changes to accept from patron modification requests
- Add ability to edit values on patron modification requests
- Add feature to question changes suggested through patron modification requests
Comment 2 Christopher Brannon 2019-08-02 15:08:20 UTC
Thanks for the comments.  I broke out the first two, added a new one, and put them in the see also.  I am leaving off the last idea, as it would be covered in 23417.  I think keeping those together makes sense.
Comment 3 Séverine Queune 2020-10-26 10:19:08 UTC Comment hidden (obsolete)
Comment 4 Séverine Queune 2020-10-26 10:19:15 UTC Comment hidden (obsolete)
Comment 5 Séverine Queune 2020-10-26 10:19:22 UTC Comment hidden (obsolete)
Comment 6 Séverine Queune 2020-10-26 10:19:28 UTC Comment hidden (obsolete)
Comment 7 Séverine Queune 2020-10-26 10:19:32 UTC Comment hidden (obsolete)
Comment 8 Séverine Queune 2020-10-26 10:19:38 UTC Comment hidden (obsolete)
Comment 9 Christopher Brannon 2020-10-26 14:35:39 UTC
What do any of these test have to do with this ticket?  I am completely confused by the activity here.
Comment 10 Owen Leonard 2020-10-26 14:41:46 UTC
(In reply to Christopher Brannon from comment #9)
> What do any of these test have to do with this ticket?

It was bug number mixup.
Comment 11 Séverine Queune 2020-10-26 14:58:32 UTC Comment hidden (obsolete)
Comment 12 Séverine Queune 2020-10-26 15:02:18 UTC
(In reply to Owen Leonard from comment #10)
> (In reply to Christopher Brannon from comment #9)
> > What do any of these test have to do with this ticket?
> 
> It was bug number mixup.

I'm sorry... typo for Bug 20212...
Hope everything is fixed now !