-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add theme compatibility indication in configuration page #15
Add theme compatibility indication in configuration page #15
Conversation
@@ -141,6 +141,7 @@ public function getContent() | |||
'type' => 'switch', | |||
'label' => $this->trans('Display email address', array(), 'Admin.Actions'), | |||
'name' => 'PS_CONTACT_INFO_DISPLAY_EMAIL', | |||
'desc' => $this->trans('Your theme needs to be compatible with this feature', array(), 'Modules.Contactinfo.Admin'), | |||
'values' => array( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @LouiseBonnard , I was not so sure of which translation domain to use here ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this PR add a configuration page to the 'Contact Information' module? I don't think there is any currently. If yes, you got the right domain. ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we added a configuration page in another PR, and this translation will be there, it just hasn't been released yet.
Good then ! thanx
I'm not sure this needs QA, does it ? 🤔 |
I don't think so either (and I checked the visual with @marionf) |
The possibility to hide/display the merchant's email address has to be compatible with the current theme. This PR indicates this below the configuration field.