| Summary: | Patron modification request page: Add link to patron record | ||
|---|---|---|---|
| Product: | Koha | Reporter: | paxed <pasi.kallinen> |
| Component: | Patrons | Assignee: | Katrin Fischer <katrin.fischer> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | gmcharlt, justinvos, kyle.m.hall, lari.taskula, tomascohen, veron |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: |
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12028 http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15195 |
||
| Change sponsored?: | --- | Patch complexity: | Small patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Bug Depends on: | 13349 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 12352: Patron modification request page: Add link to patron record
[Signed-off] Bug 12352: Patron modification request page: Add link to patron record Bug 12352: Patron modification request page: Add link to patron record |
||
|
Description
paxed
2014-06-03 04:30:47 UTC
I've started working on this bug. Justin Vos I'm not sure what is required here. While editing a patron's details in the staff client, there is already a link back to the patron's details page in the left side-bar named "Details". Is the link needed somewhere else? Hi Justin, try the following: - go to the patron account from the OPAC - enter an address change request from the details page there - see that a note appears in the patron module and on the start page of the staff client - the entry asking to moderate the modification is the one to change Created attachment 40053 [details] [review] Bug 12352: Patron modification request page: Add link to patron record The patron modification request page in the staff client does show the name and home library of a patron, but there is no way to go to the patron record. This patch adds a link to the patron details. Extras: changes display of patron name, so a missing firstname won't result in an extra , showing. To test: - Make some update requests from different patron accounts in the OPAC - Go to the patron modification request page in staff - Verify a link 'Patron details' now shows on each entry and works correctly Created attachment 40059 [details] [review] [Signed-off] Bug 12352: Patron modification request page: Add link to patron record The patron modification request page in the staff client does show the name and home library of a patron, but there is no way to go to the patron record. This patch adds a link to the patron details. Extras: changes display of patron name, so a missing firstname won't result in an extra , showing. To test: - Make some update requests from different patron accounts in the OPAC - Go to the patron modification request page in staff - Verify a link 'Patron details' now shows on each entry and works correctly Followed test plan. Works as expected. QA tools OK. Signed-off-by: Marc Véron <veron@veron.ch> Created attachment 40191 [details] [review] Bug 12352: Patron modification request page: Add link to patron record The patron modification request page in the staff client does show the name and home library of a patron, but there is no way to go to the patron record. This patch adds a link to the patron details. Extras: changes display of patron name, so a missing firstname won't result in an extra , showing. To test: - Make some update requests from different patron accounts in the OPAC - Go to the patron modification request page in staff - Verify a link 'Patron details' now shows on each entry and works correctly Followed test plan. Works as expected. QA tools OK. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org> Not sure the "Patron details" link is at the best place, but I don't have something better to suggest. Agreed - but as the name in the tab is already a link for the 'accordeon' effect, there was no other obvious spot I could think of. Patch pushed to master. Thanks Katrina! |