Skip to content
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

[NEW] Make ldap avatar source field customizable #12958

Merged
merged 11 commits into from
Jun 19, 2020

Conversation

alexbartsch
Copy link
Contributor

[NEW] Make ldap avatar source field customizable

Closes #8835

Just added a new input field called User Avatar Field to LDAP/SYNC configuration. While syncing ldap users, we should look for a user property with this name. If there is none, use thumbnailPhoto and jpegPhoto as before.

fix-8835

@alexbartsch alexbartsch changed the title Make ldap avatar source field customizable [NEW] Make ldap avatar source field customizable Dec 14, 2018
@maxdwit
Copy link

maxdwit commented Dec 18, 2018

@traxanos can you review this?

@alexbartsch
Copy link
Contributor Author

alexbartsch commented Jan 14, 2019

@sampaiodiego Any ideas on how to make this request visible for some reviewers?

@ankar84
Copy link

ankar84 commented Nov 14, 2019

We need that PR a lot, because from thumbnailPhoto we get a very low quality avatars in Rocket.Chat.
But we have a custom AD attribute with good quality photo and we like to use it.
So, please merge that PR.

@CLAassistant
Copy link

CLA assistant check
All committers have signed the CLA.

@rodrigok rodrigok added this to the 3.4.0 milestone May 29, 2020
@rodrigok rodrigok self-assigned this May 29, 2020
# Conflicts:
#	packages/rocketchat-i18n/i18n/de-IN.i18n.json
#	packages/rocketchat-i18n/i18n/de.i18n.json
#	packages/rocketchat-ldap/server/settings.js
#	packages/rocketchat-ldap/server/sync.js
@rodrigok rodrigok merged commit 5653a09 into RocketChat:develop Jun 19, 2020
@sampaiodiego sampaiodiego mentioned this pull request Jun 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

make ldap avatar source field customizable
6 participants