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

in 0.9 RC1 LDAP usr name is shown instead of display name #402

Closed
herrmannsdorfer opened this issue Oct 31, 2018 · 5 comments
Closed

in 0.9 RC1 LDAP usr name is shown instead of display name #402

herrmannsdorfer opened this issue Oct 31, 2018 · 5 comments
Labels
Milestone

Comments

@herrmannsdorfer
Copy link

What is going wrong?

Describe the bug

Testing 0.9 RC1 I found that LDAP users are shown with their usr name as participants. In 0.8.3 they were listed with their display name. LDAP usr names are not human readable - at least when pulled from a Windows AD ... Screenshot attached.

To Reproduce
Steps to reproduce the behavior:

  1. Log on to NC as an LDAP usr
  2. Create a new or edit an existing poll
  3. Yourself as a participant or as owner is shown with user name

Expected behavior
Display name should be shown as in 0.8.3

Screenshots
bildschirmfoto 2018-10-31 um 09 46 18

Information about your polls installation

Polls version? (see apps page)
0.9.0 RC1

Fresh installation or update from a prior version (from which one)?
"updated" from 0.8.3

How did you install this version?(Appstore or describe installation)
git clone https://github.com/nextcloud/polls.git

Information about your Instance of Nextcloud/ownCloud

Nextcloud or ownCloud?
NC 14.0.3

@dartcafe
Copy link
Collaborator

I will look after it. Seems, that we display the user id instead of the user's full name.

@dartcafe dartcafe added the bug label Nov 9, 2018
@dartcafe dartcafe added this to the 0.9.0 milestone Nov 9, 2018
dartcafe added a commit that referenced this issue Nov 10, 2018
Changings to shareDiv
#402
@dartcafe
Copy link
Collaborator

Did a quick fix displaying the user's DisplayName.

@dartcafe
Copy link
Collaborator

@herrmannsdorfer Can you confirm?
RC 2 is out.

@herrmannsdorfer
Copy link
Author

Sorry, latest polls app from git repo downloaded a few minutes ago still shows user id instead of display name ...

@herrmannsdorfer
Copy link
Author

Sorry again. I used the wrong branch. Bug is fixed in develop-0.9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants