Skip to content

Conversation

@wankdanker
Copy link
Contributor

Some NC installations may use a LDAP UUID for the username. If that
is the case, then the tooltip for who a file/folder was shared by may say something
like Shared with Steve McQueen (smqueen@yada.com) by FKDJ39-38D794JG-4398GJODF9H3G-4534.

This commit changes the value showed for the owner to the full display name.

@rullzer rullzer added 3. to review Waiting for reviews enhancement labels Feb 12, 2021
@rullzer rullzer added this to the Nextcloud 22 milestone Feb 12, 2021
@rullzer
Copy link
Member

rullzer commented Feb 12, 2021

/compile amend /

Some NC installations may use a LDAP UUID for the username. If that
is the case, then the tooltip for who a file/folder was shared by may say something
like `Shared with Steve McQueen (smqueen@yada.com) by FKDJ39-38D794JG-4398GJODF9H3G-4534`.

This commit changes the value showed for the owner to the full display name.

Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
@npmbuildbot-nextcloud npmbuildbot-nextcloud bot force-pushed the files_sharing_owner_display_name branch from 165645b to 1f47ae7 Compare February 12, 2021 07:32
@rullzer rullzer merged commit 9929add into nextcloud:master Feb 15, 2021
@welcome
Copy link

welcome bot commented Feb 15, 2021

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@rullzer
Copy link
Member

rullzer commented Feb 15, 2021

/backport to stable21

@rullzer
Copy link
Member

rullzer commented Feb 15, 2021

/backport to stable20

@rullzer
Copy link
Member

rullzer commented Feb 15, 2021

/backport to stable19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants