-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Auth Twig UI (Part 1) #16951
Auth Twig UI (Part 1) #16951
Conversation
91b0dd2
to
74d6e8b
Compare
}) }} | ||
|
||
{{ fields.dropdownYesNo('use_bind', item.fields['use_bind'], __('Use bind'), { | ||
helper: __("Indicates whether a simple bind operation should be used during connection to LDAP server. Disabling this behaviour can be required when LDAPS bind is used.")|e('html') |
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.
The helper is already in the target macro.
helper: __("Indicates whether a simple bind operation should be used during connection to LDAP server. Disabling this behaviour can be required when LDAPS bind is used.")|e('html') | |
helper: __("Indicates whether a simple bind operation should be used during connection to LDAP server. Disabling this behaviour can be required when LDAPS bind is used.") |
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.
The field helpers allow HTML content, but we wouldn't want to accidentally add HTML content from a localization.
|
||
{{ fields.textField('login_field', item.fields['login_field'], __('Login field'), field_options) }} | ||
{{ fields.textField('sync_field', item.fields['sync_field'], __('Synchronization field'), { | ||
helper: __('Synchronization field cannot be changed once in use.')|e('html'), |
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.
helper: __('Synchronization field cannot be changed once in use.')|e('html'), | |
helper: __('Synchronization field cannot be changed once in use.'), |
Co-authored-by: Cédric Anne <cedric.anne@gmail.com>
See changelog for intended changes/removals/fixes.