the.rampage.rado Posted November 26 Posted November 26 I don't see the second module that was mentioned with the 1.6 release, I only have Contact Form IP address blocker.
datakick Posted November 26 Posted November 26 2 hours ago, the.rampage.rado said: @datakickย little bit on a tangent, but I don't see the second module that was mentioned with the 1.6 release, I only have Contact Form IP address blocker. Indeed. I fixed this, you should see the other module as well now
the.rampage.rado Posted November 26 Author Posted November 26 Thank you! Will those filters work or I did not set them correctly?
datakick Posted November 26 Posted November 26 3 minutes ago, the.rampage.rado said: Thank you! Will those filters work or I did not set them correctly? yes, they should. Try to send customer message using email address with .ru email ๐
the.rampage.rado Posted November 26 Author Posted November 26 7 minutes ago, datakick said: yes, they should. Try to send customer message using email address with .ru email ๐ I don't want to register such account. ๐ Thank you!
datakick Posted November 26 Posted November 26 Just now, the.rampage.rado said: I don't want to register such account. ๐ You don't need to ๐ย the email address is entered on contact form, for non-logged in customers
the.rampage.rado Posted November 26 Author Posted November 26 Of course... ๐ The rule is working as expected!
the.rampage.rado Posted November 26 Author Posted November 26 The error message 'Your email address is blocked from sending messages through contact form. Please send email to xxx instead.' is currently not translatable. I'll split this offtopic later.
datakick Posted November 26 Posted November 26 Indeed. Looks like a bug in translation system, when $this->l() does not accept strings with quotes, only with single apostrophes. Oh my... The fix is here:ย https://github.com/thirtybees/tbcontactformrecipientfilter/commit/0318e7a56d261a21e92fc9960aff6ed3bd80b66d The fix of the actual bug (translation strings extraction in core) will be hopefully in next version. 1
the.rampage.rado Posted November 26 Author Posted November 26 Thanks! Tomorrow I can make PRs for the modules that have the same 'issue'.
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now