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

Katrin Fischer <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #15 from Katrin Fischer <[email protected]> ---
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.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
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/

Reply via email to