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

Show username in collaborators/share tab #1190

Closed
butonic opened this issue Jan 8, 2021 · 14 comments · Fixed by cs3org/reva#1451
Closed

Show username in collaborators/share tab #1190

butonic opened this issue Jan 8, 2021 · 14 comments · Fixed by cs3org/reva#1451
Labels
Category:Defect Existing functionality is not working as expected Type:Bug

Comments

@butonic
Copy link
Member

butonic commented Jan 8, 2021

ocis.owncloud.works does not show the email or username in the list of recipients.

@refs
Copy link
Member

refs commented Jan 12, 2021

@butonic can you be more specific? Where exactly? :)

@refs refs added bug and removed enhancement labels Jan 12, 2021
@settings settings bot removed the bug label Jan 12, 2021
@refs refs added Category:Feature Category:Defect Existing functionality is not working as expected and removed Category:Feature labels Jan 13, 2021
@exalate-issue-sync
Copy link

Willy Kloucek commented: I guess here:
image.png

If you have it shared with two "Max Mustermann" you should be able to distinguish between them.

@butonic butonic self-assigned this Feb 1, 2021
@butonic
Copy link
Member Author

butonic commented Feb 3, 2021

Hm this should be working as of nov last year: owncloud/web#4203, but I cannot see the email on demo.owncloud.com when trying to find recipients or when accepting shares ... 🤔 @kulmann @LukasHirt any idea?

@butonic butonic removed their assignment Feb 3, 2021
@LukasHirt
Copy link
Contributor

but I cannot see the email on demo.owncloud.com when trying to find recipients or when accepting shares

@butonic Check if it's enabled in the settings. Admin -> Sharing -> Extra field to display in autocomplete results. When it's enabled, I can see it just fine:

Screenshot 2021-02-04 at 1 49 07

Screenshot 2021-02-04 at 1 49 17

Screenshot 2021-02-04 at 1 49 23

@kulmann
Copy link
Member

kulmann commented Feb 4, 2021

ownCloud Web simply renders whatever value shareWithAdditionalInfo of a sharee holds.
Doing a quick git grep "shareWithAdditionalInfo" in the reva and ocis repo shows 0 results.

See relevant ownCloud 10 code here:
https://github.com/owncloud/core/blob/39b3dd7627465eb316d1b707bd21cb0b6b3c972e/apps/files_sharing/lib/Controller/ShareesController.php#L206
https://github.com/owncloud/core/blob/39b3dd7627465eb316d1b707bd21cb0b6b3c972e/apps/files_sharing/lib/Controller/ShareesController.php#L278

@butonic
Copy link
Member Author

butonic commented Feb 12, 2021

fix landed in ocis master with #1588

@butonic butonic closed this as completed Feb 12, 2021
@labkode
Copy link
Contributor

labkode commented Feb 19, 2021

Can we have it configurable? choose between username and mail. Maybe have a template? like storage user layout?

@labkode
Copy link
Contributor

labkode commented Mar 12, 2021

@butonic

@butonic butonic reopened this Mar 12, 2021
@butonic
Copy link
Member Author

butonic commented Mar 12, 2021

currently hardcoded to email, needs to be made configurable to at least username or email. best would be a template.

@labkode
Copy link
Contributor

labkode commented Mar 12, 2021

Same template mechanism we use for the storage layout, that would be great

@labkode
Copy link
Contributor

labkode commented Mar 26, 2021

@butonic when you're back let's catch up on this

@ishank011
Copy link
Contributor

Done cs3org/reva#1588

@settings settings bot removed the p2-high label Apr 7, 2021
@pascalwengerter
Copy link
Contributor

What's the status quo on this one?

@ishank011
Copy link
Contributor

@pascalwengerter this can be closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Defect Existing functionality is not working as expected Type:Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants