https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26067 Amit Gupta <amitddng135@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |ASSIGNED --- Comment #16 from Amit Gupta <amitddng135@gmail.com> --- (In reply to Katrin Fischer from comment #15)
Hi Amit,
1) sysprefs.sql is sorted alphabetically, can you please move the new pref line into the right spot?
2) opac.pref Doesn't quite work yet as a sentence and there is a little typo on 'no'. Maybe:
Now: + - pref: PatronEmailValidationMXrecord + choices: + yes: Require + no: "Dont't require" + - "Mail Exchange (MX) records are DNS records that are necessary for delivering email to Patron." + -
Suggestion: + - pref: PatronEmailValidationMXrecord + choices: + yes: Require + no: "Don't require" + - "email address entered on patron self registration to pass Mail Exchange (MX) record check."
3) As we allow to use every patron email address as the 'to' address we sent email to (see AutoEmailPrimaryAddress), I'd suggest to make this change for all the email address fields.
Thank Katrin for QA will do the changes. -- You are receiving this mail because: You are watching all bug changes.