https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30237
Katrin Fischer <katrin.fisc...@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA Keywords| |Manual, | |release-notes-needed --- Comment #11 from Katrin Fischer <katrin.fisc...@bsz-bw.de> --- Hi all, almost there! 1) This includes a database update to insert a new letter. I am not sure about this approach. I think it would be better if we changed the letter code on the existing ACCTDETAILS instead, because it will cause less surprising behaviour. The existing notice will kept to be sent for libraries using the feature instead of it suddently changing text and turning English. (blocker) 2) We could add a note to the system preference about the letter code, so people will be able to see the change a little better. (suggestion) 3) QA test tools: FAIL installer/data/mysql/atomicupdate/bug_30237.pl FAIL file permissions File must have the exec flag 4) New letter Thank you for doing this, I already had filed a separate bug before stumbling onto this here! Some small stuff and hopefully good suggestions: a) Tiny typo: Thank you + - "Thankyou for joining [% Koha.Preference('LibraryName') %]" b) Linking 'here' Never link "here", it's a known accessibility issue with screen readers, but also because we are all used to scan for links now and it doesn't give an useful clue on where it leads. "The library's catalog can be found <a href='[% Koha.Preference('OPACBaseURL') %]'>here</a>." Better to say something like: You can search for all our materials in our <a href='[% Koha.Preference('OPACBaseURL') %]'>catalog</a>." And patron's won't know that this is Koha or who the administrator is, better to ask them to get in touch with the library staff :) If you have any problems or questions regarding your account, please contact your Koha Administrator. Better: If you have any problems or questions regarding your account, please contact your library. 5) French installer As we don't know yet if the French installer will be gone in time for release, we should update the installer with the new notice: installer/data/mysql/fr-FR/1-Obligatoire/sample_notices.sql -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/