Bug 26724

Summary: Improve link text for downloading the CSV file on patron import page
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: TemplatesAssignee: Lisette Scheer <lisettepalouse+koha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: aleisha, david, jonathan.druart, lisettepalouse+koha, oleonard
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This enhancement removes the "here" link for the example CSV file for importing patrons (Tools > Patrons and circulation > Import patrons). It now reads "Download a Starter CSV file with all the columns. ...", with "Starter CSV file" now linked.
Version(s) released in:
20.11.00
Bug Depends on:    
Bug Blocks: 26722    
Attachments: Attachment to Bug 26724 - Improve link text for downloading the CSV file on patron import page
Followup patch to further improve link text for downloading the CSV file on the patron import page.
Attachment to Bug 26724 - Improve link text for downloading the CSV file on patron import page
Attachment to Bug 26724 - Improve link text for downloading the CSV file on patron import page
Bug: Improve link text for downloading the CSV file on patron import page.
Bug 26724: Improve link text for downloading the CSV file on patron import page.

Description Katrin Fischer 2020-10-18 15:32:41 UTC
On Tools > Patron import there is a link labelled "here" on the right side that should be improved (see depending bug 26722).
Comment 1 Katrin Fischer 2020-10-21 03:40:31 UTC
*** Bug 26759 has been marked as a duplicate of this bug. ***
Comment 2 Lisette Scheer 2020-10-24 17:58:46 UTC
Created attachment 112389 [details] [review]
Attachment to Bug 26724 - Improve link text for downloading the CSV file on patron import page

Bug 26724: Improve link text for downloading the CSV file on patron import page

On tools > Patron Import there is a link labeled "here" on the right
side that should be more descriptive. This patch updates the link.

To test:
1. Go to tools
2. Select "Import Patrons"
3. On the right side of the screen, observe the "here" link in the
second list item.
4. Apply patch
5. Repeat steps 1-2
6. Observe the updated link.
7. Sign off
Comment 3 David Nind 2020-10-25 05:21:50 UTC
I would suggest removing "here". 

That is, just say "Download a starter CSV file with all the columns."
Comment 4 Katrin Fischer 2020-10-25 10:36:31 UTC
If you look at the examples on the main bug I think we should maybe also remove "Download" and just have "Starter CSV file" or "Sample CSV file" in the link text.
Comment 5 David Nind 2020-10-25 10:58:10 UTC
+1
Comment 6 Lisette Scheer 2020-10-25 17:31:50 UTC Comment hidden (obsolete)
Comment 7 Lisette Scheer 2020-10-25 17:39:22 UTC
Created attachment 112479 [details] [review]
Attachment to Bug 26724 - Improve link text for downloading the CSV file on patron import page

Bug: 26724: Improve  link text for downloading the CSV file on patron import page.

On tools > Patron Import there is a link labeled "here" on the right
side that should be more descriptive. This patch updated the link.

To test:
1. Go to tools
2. Under patrons select "Import Patrons"
3. On the right side of the screen, observe the "here" link in the
second list item.
4. Apply patch.
5. Reload page.
6. Obsesrve the updated link text.
7. Sign off
Comment 8 David Nind 2020-10-25 19:17:50 UTC
The patch no longer applies 8-(

root@kohadevbox:koha(master)$ git bz apply 26724
Bug 26724 - Improve link text for downloading the CSV file on patron import page

112479 - Attachment to Bug 26724 - Improve link text for downloading the CSV file on patron import page

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug: 26724: Improve link text for downloading the CSV file on patron import page.
error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt).
error: could not build fake ancestor
Patch failed at 0001 Bug: 26724: Improve link text for downloading the CSV file on patron import page.
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem run "git bz apply --continue".
If you would prefer to skip this patch, instead run "git bz apply --skip".
To restore the original branch and stop patching run "git bz apply --abort".
Patch left in /tmp/Attachment-to-Bug-26724---Improve-link-text-for-do-JLSMdI.patch
Comment 9 Lisette Scheer 2020-10-27 15:44:27 UTC
Created attachment 112582 [details] [review]
Attachment to Bug 26724 - Improve link text for downloading the CSV file on patron import page

Bug: Improve link text for downloading the CSV file on patron import page.

On tools > Patron Import there is a a link labelled "here" on the right
side that should be more descriptive. This patch updates the link.

To test:
1. Go to tools
2. Under patrons select "Import Patrons"
3. On the right side of the screen, observe the "here" link in the
second list item.
4. Apply patch.
5. Reload page
6. Observe the updated link text.
7. Sign off
Comment 10 David Nind 2020-10-27 17:56:53 UTC
Created attachment 112586 [details] [review]
Bug: Improve link text for downloading the CSV file on patron import page.

On tools > Patron Import there is a a link labelled "here" on the right
side that should be more descriptive. This patch updates the link.

To test:
1. Go to tools
2. Under patrons select "Import Patrons"
3. On the right side of the screen, observe the "here" link in the
second list item.
4. Apply patch.
5. Reload page
6. Observe the updated link text.
7. Sign off

https://bugs.koha-community.org/show_bug.cgi?id=26724

Signed-off-by: David Nind <david@davidnind.com>
Comment 11 Katrin Fischer 2020-11-01 13:12:27 UTC
Created attachment 112802 [details] [review]
Bug 26724: Improve link text for downloading the CSV file on patron import page.

On tools > Patron Import there is a a link labelled "here" on the right
side that should be more descriptive. This patch updates the link.

To test:
1. Go to tools
2. Under patrons select "Import Patrons"
3. On the right side of the screen, observe the "here" link in the
second list item.
4. Apply patch.
5. Reload page
6. Observe the updated link text.
7. Sign off

https://bugs.koha-community.org/show_bug.cgi?id=26724

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 12 Jonathan Druart 2020-11-02 10:05:51 UTC
Pushed to master for 20.11, thanks to everybody involved!