Polls: Provide displayNames instead of userId

[herrmannsdorfer]

402 is back.

[herrmannsdorfer]

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]

I have to fiddle out, how to get the displaynames via the js API. comes later.

[tortuetorche]

Hi @dartcafe,

First of all, thank you for this great Nextcloud App 👍

Here some clues to display name instead of user id, based on Nextcloud Talk/Spreed code:
Nextcloud Talk use the @nextcloud/auth NPM module.
Dependency reference: https://github.com/nextcloud/spreed/blob/v8.0.0/package.json#L18

Then, the code to load this module: https://github.com/nextcloud/spreed/blob/v8.0.0/src/PublicShareAuthSidebar.vue#L37

The code to manipulate the getCurrentUser() object: https://github.com/nextcloud/spreed/blob/v8.0.0/src/PublicShareAuthSidebar.vue#L164

Finally, the code to extract the displayName of the user, if any: https://github.com/nextcloud/spreed/blob/v8.0.0/src/store/actorStore.js#L135

Have a good day,
Tortue Torche

[tortuetorche]

Hi folks,

I think my previous comment go in the wrong direction, because it's only focused on the current user 😕

Cheers,
Tortue Torche

[dartcafe]

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.

[Dennis1993]

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

[dartcafe]

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

[herrmannsdorfer]

Resolved in 1.2.0b4. Good job!