| Summary: | Add vendor to the acq tab on the details page in staff client | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
| Component: | Acquisitions | Assignee: | Katrin Fischer <katrin.fischer> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | jonathan.druart, tomascohen |
| 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: | ||
| Circulation function: | |||
| Attachments: |
Bug 13962: Add vendor to acq details tab in staff
Bug 13962: Add vendor to acq details tab in staff Bug 13962: Add vendor to acq details tab in staff Bug 13962: Add link to the vendor detail page |
||
|
Description
Katrin Fischer
2015-04-07 14:38:41 UTC
Created attachment 40050 [details] [review] Bug 13962: Add vendor to acq details tab in staff Implementing some feedback from our user meeting: The acquisition details tab on the detail page in staff should also show the vendor of the order. To test: - Make sure AcquisitionDetails is active. - Create an order or look up an order in the acqusition module. - Go to the ordered record and check the 'Acquisition details' tab - Verify the vendor shows up there as first column now - Check that sorting and display of the other columns are still working correctly Note: Also fixes a </th> that should be a </td> Created attachment 40056 [details] [review] Bug 13962: Add vendor to acq details tab in staff Implementing some feedback from our user meeting: The acquisition details tab on the detail page in staff should also show the vendor of the order. To test: - Make sure AcquisitionDetails is active. - Create an order or look up an order in the acqusition module. - Go to the ordered record and check the 'Acquisition details' tab - Verify the vendor shows up there as first column now - Check that sorting and display of the other columns are still working correctly Note: Also fixes a </th> that should be a </td> Signed-off-by: Aleisha <aleishaamohia@hotmail.com> Created attachment 40192 [details] [review] Bug 13962: Add vendor to acq details tab in staff Implementing some feedback from our user meeting: The acquisition details tab on the detail page in staff should also show the vendor of the order. To test: - Make sure AcquisitionDetails is active. - Create an order or look up an order in the acqusition module. - Go to the ordered record and check the 'Acquisition details' tab - Verify the vendor shows up there as first column now - Check that sorting and display of the other columns are still working correctly Note: Also fixes a </th> that should be a </td> Signed-off-by: Aleisha <aleishaamohia@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org> Created attachment 40193 [details] [review] Bug 13962: Add link to the vendor detail page This patch 1/ uses the class of the th to filter the columns and 2/ adds a link on the vendor name to the vendor detail page. Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org> Katrin, could you quickly confirm that you agree with my follow-up? (In reply to Jonathan Druart from comment #5) > Katrin, could you quickly confirm that you agree with my follow-up? I was not expected a quick answer, but just to have a quick look :) Patches pushed to master. Thanks Katrina! |