The --since option is not documented at all in the script borrowers-force-messaging-defaults.pl. --since Will only update borrowers enrolled since the specified date. Examples: --since "2022-07-13" --since `date -d "1 day ago" '+%Y-%m-%d'
Created attachment 137724 [details] [review] Bug 31155: Document --since option in help of borrowers-force-messaging-defaults.pl This patch adds a section about the --since option in the help section of borrowers-force-messaging-defaults.pl To test: 0) Apply patch 1) run the script with the --help option, e.g. ./misc/maintenance/borrowers-force-messaging-defaults.pl --help --> There should be an explanation of the --since option with examples for specific and relative dates 2) Optionally, run the script with different options --> The behaviour shouldn't have changed
I copied the syntax from the other options, I hope it's ok. I've never edited .pl before...
Created attachment 138763 [details] [review] Bug 31155: Document --since option in help of borrowers-force-messaging-defaults.pl This patch adds a section about the --since option in the help section of borrowers-force-messaging-defaults.pl To test: 0) Apply patch 1) run the script with the --help option, e.g. ./misc/maintenance/borrowers-force-messaging-defaults.pl --help --> There should be an explanation of the --since option with examples for specific and relative dates 2) Optionally, run the script with different options --> The behaviour shouldn't have changed Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Current patch-complexity: Trivial patch
Going straight to PQA for this documentation fix.
Pushed to master for 22.11. Nice work everyone, thanks!
Backported to 22.05.x for 22.05.06
Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed. Maybe docs to update.
Thanks! Pushed to 21.11 for 21.11.12
Seems to be already well documented in the manual. I think I did it in the manual first before this patch.