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 EVENTs to get additionals user attributes from LDAP and show them #1660
base: master
Are you sure you want to change the base?
Add EVENTs to get additionals user attributes from LDAP and show them #1660
Conversation
Plugin exemple for use : https://github.com/Association-Cocktail/LdapCacheFields |
@vboctor |
191a3a2
to
5aa5040
Compare
c8a68ff
to
298787a
Compare
c2b75b1
to
6328e50
Compare
6328e50
to
791b759
Compare
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.
-
For attributes to display the plugin event should return logical data that is rendered by core. We have been moving away from have events echo html output. Reason is that it is not API friendly and breaks whenever Mantis UI is changed.
-
For the extra LDAP fields, why not just use a config option rather than a plugin event?
-
Should the extra LDAP info be included in the REST API that retrieves user info? eg in ldap_fields element?
I think I can make that change.
I used a plugin event following a past suggestion but I'll use the method you prefer ...
Probably, it makes sense. |
791b759
to
6ca6c9b
Compare
No description provided.