Bug 37538 - Improve documentation printed by connexion_import_daemon.pl --help
Summary: Improve documentation printed by connexion_import_daemon.pl --help
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Command-line Utilities (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Rudolf Byker
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 37539
  Show dependency treegraph
 
Reported: 2024-07-31 14:35 UTC by Rudolf Byker
Modified: 2024-08-01 23:43 UTC (History)
4 users (show)

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


Attachments
Improve output of `connexion_import_daemon.pl --help` (10.52 KB, patch)
2024-07-31 14:52 UTC, Rudolf Byker
Details | Diff | Splinter Review
Bug 37538: Improve documentation for the OCLC import daemon script. (10.33 KB, patch)
2024-08-01 00:12 UTC, David Cook
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Rudolf Byker 2024-07-31 14:35:36 UTC
Patch incoming :)

See related discussions at:

- https://chat.koha-community.org/koha-community/pl/o18q57a1pt879dgp63ujtcajpe
- https://chat.koha-community.org/koha-community/pl/gz3mjusns3bdme3j4f8sp83sfw
- https://chat.koha-community.org/koha-community/pl/yzxmrfegcbnrjgqgjt7t6wjwhy

I plan to follow this up with a patch for the koha-manual, which I will also link here.
Comment 1 Rudolf Byker 2024-07-31 14:52:42 UTC
Created attachment 169894 [details] [review]
Improve output of `connexion_import_daemon.pl --help`

Patch attached.
Comment 2 David Cook 2024-08-01 00:12:50 UTC
Created attachment 169921 [details] [review]
Bug 37538: Improve documentation for the OCLC import daemon script.

To test:
1) Run `/usr/share/koha/bin/connexion_import_daemon.pl --help`
2) Read the output and check for errors, whether grammatical, spelling, or factual.

Sponsored-by: Reformational Study Centre <www.refstudycentre.com>
Signed-off-by: David Cook <dcook@prosentient.com.au>
Comment 3 David Cook 2024-08-01 00:14:57 UTC
Patch works as described.

Test plan should use "./misc/bin/connexion_import_daemon.pl --help" rather than "/usr/share/koha/bin/connexion_import_daemon.pl --help" but not a problem

I'm not sure that the format is quite inline with other Koha scripts... but it looks like a very useful change. I haven't verified the truth of all the text changes, but they look good at a glance, and much more helpful than the current help text.
Comment 4 Rudolf Byker 2024-08-01 03:04:56 UTC
I'm not sure what you mean by the format not being in line with other scripts. I'm willing to improve it. I'm also willing to just let it go, and eager to get it merged, so that I can refer users to the script's --help output from the manual.
Comment 5 David Cook 2024-08-01 03:39:34 UTC
(In reply to Rudolf Byker from comment #4)
> I'm not sure what you mean by the format not being in line with other
> scripts. 

Say in comparison to the scripts in /usr/sbin/koha-*

> I'm willing to improve it. I'm also willing to just let it go, and
> eager to get it merged, so that I can refer users to the script's --help
> output from the manual.

I think that we can just let it go. Honestly, I don't think I have any specific recommendations on how to improve it either. A QAer might have some thoughts on this one, but with a script like this which is so rarely used... I think the more documentation the better in any case.
Comment 6 David Nind 2024-08-01 21:22:23 UTC
I changed the status to "Needs documenting".

Because of the way the Koha workflow is set up in Bugzilla (aimed at changes to the Koha code base), I needed to change the status a few times to get the right status shown:
1. RESOLVED FIXED
2. REOPENED
3. Needs documenting

The status can be changed to RESOLVED FIXED once the merge request goes through.
Comment 7 David Cook 2024-08-01 23:27:45 UTC
(In reply to David Nind from comment #6)
> I changed the status to "Needs documenting".
>
> The status can be changed to RESOLVED FIXED once the merge request goes
> through.

What needs documenting for this one? Is it just for the release notes or something else? this change shouldn't affect the manual?
Comment 8 David Nind 2024-08-01 23:43:05 UTC
(In reply to David Cook from comment #7)
> (In reply to David Nind from comment #6)
> > I changed the status to "Needs documenting".
> >
> > The status can be changed to RESOLVED FIXED once the merge request goes
> > through.
> 
> What needs documenting for this one? Is it just for the release notes or
> something else? this change shouldn't affect the manual?

OOPS, I updated the wrong bug! (Should have been bug 37539!)

Thanks for spotting, David.

Have changed back to Signed Off....