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.

Sign up for GitHub

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

Jump to bottom

#187 Populate avatar from SAML attribute #640

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

#187 Populate avatar from SAML attribute #640

wants to merge 2 commits into from

Conversation

Lumrenion
Copy link

This PR addresses #187.

  • In admin configuration, a new configuration field avatar_mapping is added.
  • If avatar_mapping configuration field is not empty, the user wont be able to upload an image in the nextcloud profile settings
  • The avatar_mapping field holds a publicly available URL to the profile image of the user. As usual, the image will be downloaded to nextcloud when the user logs in via user_saml.

@Lumrenion Lumrenion mentioned this pull request Aug 3, 2022
Closed
@AkarinServer
Copy link

What time can this PR be merged into main branch?

@solracsf
Copy link
Member

@Lumrenion conflicts and DCO need to be solved. Thanks 👍

@cb0s

This comment was marked as spam.

Sign in to view
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
2. developing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Lumrenion @AkarinServer @solracsf @cb0s