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

Provide displayNames instead of userId #715

Closed
herrmannsdorfer opened this issue Jan 8, 2020 · 8 comments
Closed

Provide displayNames instead of userId #715

herrmannsdorfer opened this issue Jan 8, 2020 · 8 comments
Labels
enhancement
Milestone
1.3

Comments

@herrmannsdorfer
Copy link

#402 is back.

Bildschirmfoto 2020-01-08 um 13 42 17
@herrmannsdorfer
Copy link
Author

Looking at the screenshot I would also take care of the column width (created entries overlapping expiration date) and the vertical alignment of the owner to the other text lines.

@dartcafe

This comment has been minimized.

Sign in to view
@dartcafe dartcafe changed the title 1.0b2: LDAP user id instead of DisplayName is shown Provide displayNames instead of userId Jan 8, 2020
@dartcafe dartcafe added this to the next milestone Jan 8, 2020
@tortuetorche

This comment has been minimized.

Sign in to view
@dartcafe dartcafe mentioned this issue Jan 25, 2020
Closed
@tortuetorche

This comment has been minimized.

Sign in to view
@dartcafe
Copy link
Collaborator

Yes, right. The displayNames are AFAIK not available via the JS API. But I want to avoid loading every username via a individual API-Call. That means, I have to load them when gathering the information in the backend. This means extending the models. I need some time for that. Stabilizing the app has a higher priority ATM.

This was referenced Jan 25, 2020
Closed
Closed
@Dennis1993
Copy link

I have the same problem, I don’t know who has voted 😅🤷‍♂️
NC 17.0.2 on Ubuntu with Polls 1.1.6

dartcafe added a commit that referenced this issue Jan 29, 2020
@dartcafe
Add displayname for LDAP users #715
601c8f1
@herrmannsdorfer herrmannsdorfer mentioned this issue Jan 29, 2020
Closed
@dartcafe dartcafe mentioned this issue Jan 29, 2020
Merged
@dartcafe
Copy link
Collaborator

See beta4 in the releases. There are some more enhancements and fixes. please report bugs in new issues.

@herrmannsdorfer
Copy link
Author

Resolved in 1.2.0b4. Good job!

@dartcafe dartcafe modified the milestones: next, 1.3 Apr 6, 2020
@dartcafe dartcafe mentioned this issue Jun 7, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
1.3
Development

No branches or pull requests
4 participants
@Dennis1993 @tortuetorche @dartcafe @herrmannsdorfer