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

[BUG] <People peopleQueries={some string array} does not show any avatars (React) #3172

Closed
Ofer-Gal opened this issue May 8, 2024 · 2 comments · Fixed by #3284
Closed

[BUG] <People peopleQueries={some string array} does not show any avatars (React) #3172

Ofer-Gal opened this issue May 8, 2024 · 2 comments · Fixed by #3284
Assignees
Labels
Milestone

Comments

@Ofer-Gal
Copy link

Ofer-Gal commented May 8, 2024

Describe the bug
According to the docs, the peopleQueries should have a string array of email addresses like [ "user1@someDomain.onmicrosoft.com", "user2@someDomain.onmicrosoft.com", "user3@someDomain.onmicrosoft.com", ]

When it is used in the HTML like so:
<People peopleQueries={ the stateVar with string array of email addresses } It does not show anything.
I tried to use fallbackDetails with an array of [ { mail: "user1@someDomain.onmicrosoft.com", }, { mail: "user2@someDomain.onmicrosoft.com", }, { mail: "user3@someDomain.onmicrosoft.com", }, ]

I got the control to show all users of the tenant. If the wrong type is passed to it it errors with error see below in screenshots

Expected behavior
The <People component should show avatars of the selected emails in a string array.

Screenshots

image

Environment (please complete the following information):

  • OS: windows 11
  • Browser [edge]
  • Framework [react]
  • Context [SharePoint]
  • Version [4.2.1]
  • Provider [SharePoint)
    Additional context

Using <Person in a map of the same array of email addresses, works fine. But <People is more suitable for the app.

Thank you

@Mnickii
Copy link
Collaborator

Mnickii commented Sep 3, 2024

Hi @Ofer-Gal, in the repro https://stackblitz.com/edit/mgt-8925-1jlicc?file=src%2Fmain.tsx,src%2FApp.tsx,package.json, this seems to be working as expected. Are you still experiencing this issue?

@Ofer-Gal
Copy link
Author

Ofer-Gal commented Sep 3, 2024

No it is working in later version

Thank you

@Ofer-Gal Ofer-Gal closed this as completed Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done ✔️
Development

Successfully merging a pull request may close this issue.

2 participants