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

Allow assignment of images already in Nextcloud as contact photos #106

Closed
2 tasks
mossholderm opened this issue Jan 26, 2017 · 5 comments · Fixed by #1024
Closed
2 tasks

Allow assignment of images already in Nextcloud as contact photos #106

mossholderm opened this issue Jan 26, 2017 · 5 comments · Fixed by #1024
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature or request good first issue Good for newcomers medium Medium priority
Milestone

Comments

@mossholderm
Copy link

mossholderm commented Jan 26, 2017

  • If the image is already in Nextcloud, should we really need to upload it again?
  • Provide an editor to cut and resize images like we don on the user settings
@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of enhancement New feature or request labels Jan 26, 2017
@jancborchardt
Copy link
Member

@icewind1991 @nickvergessen @MorrisJobke who implemented the »Choose from Files« function for the user avatar on the personal settings page? We could reuse that here. :)

@jancborchardt jancborchardt added the medium Medium priority label Jul 4, 2017
@MorrisJobke
Copy link
Member

@icewind1991 @nickvergessen @MorrisJobke who implemented the »Choose from Files« function for the user avatar on the personal settings page? We could reuse that here. :)

I think @icewind1991 added this. The Choose from Files is OC.dialogs.filepicker() and those parameters: https://github.com/nextcloud/server/blob/a2f6fea4081e7920ed6cd33a96e2b80dc0c51303/core/js/oc-dialogs.js#L170-L178

@jancborchardt jancborchardt added the good first issue Good for newcomers label Jul 6, 2017
@jancborchardt
Copy link
Member

@MorrisJobke thanks for the pointer! :) I’m setting this as starter issue then, would you agree?

@skjnldsv
Copy link
Member

skjnldsv commented Oct 2, 2018

I edited OP to merge two issues into one, see #33

@brettferrell
Copy link

Or, once you have the email address, check for a Gravatar.

https://en.gravatar.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature or request good first issue Good for newcomers medium Medium priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants