microsoftgraph / microsoft-graph-toolkit

Authentication Providers and UI components for Microsoft Graph 🦒
https://docs.microsoft.com/graph/toolkit/overview
Other
963 stars 309 forks source link

React PeoplePicker with groupId={filterDPC} fails on any transitive Search (Not Found) #3258

Closed Ofer-Gal closed 3 months ago

Ofer-Gal commented 4 months ago

Describe the bug When not adding groupId property, the search works fine more or less. image When using with groupId property

             <PeoplePicker selectionMode="single" 
                    type='person' userType='user'
                    showMax={40} transitiveSearch={true} groupId={filterDPC} 
                    selectionChanged={onuserSelectionChanged}
              />

The list shows only people of said group but all searches fail. image image

Steps to reproduce the behavior: Just use the code with valid groupId property

Expected behavior The above code should find users with search text included in the name or email. Screenshots See above

Environment (please complete the following information):

BTW I tested your example in https://mgt.dev/?path=/story/components-mgt-people-picker-properties--group-id and it has the same issue. 😓

musale commented 3 months ago

There is a missing header (ConsistencyLevel: eventual) to calls made to the groups api with filters. Fixing.

Ofer-Gal commented 3 months ago

Next version?

musale commented 3 months ago

Next version?

Yes. v4.2.4.