microsoft-graph-toolkit icon indicating copy to clipboard operation
microsoft-graph-toolkit copied to clipboard

[BUG]: GroupType property not working as expected in PeoplePicker React

Open ridhi58 opened this issue 11 months ago • 5 comments

Describe the bug groupType property in PeoplePicker throwing error in React.

To Reproduce Steps to reproduce the behavior:

  1. Open the people picker component in React app from the storybook - https://mgt.dev/?path=/story/components-mgt-people-picker-react--people-picker.
  2. Add type="group" and groupType="distribution"
  3. try searching in searcbox, it throws error saying that @microsoft_mgt-react.js?v=03615018:2902 Uncaught (in promise) TypeError: groupTypes.join is not a function.

Expected behavior groupType should work as expected.

Screenshots

Image Image

AdditionalInformation

It doesnt allow to pass array of strings or multiple comma separated values as mentioned in the documentation. It only allows to enter one groupType in string format.

ridhi58 avatar Jan 18 '25 13:01 ridhi58

Ho @ridhi58 thank you for surfacing this. I had reproduced this in a different issue https://github.com/microsoftgraph/microsoft-graph-toolkit/issues/3358 and made fix https://github.com/microsoftgraph/microsoft-graph-toolkit/pull/3359. Please test using @microsoft/mgt@next version

musale avatar Jan 20 '25 07:01 musale

with the next version , the whole component is not working at all. It doesnt hit any api . there are no errors as well.

ridhi58 avatar Jan 20 '25 13:01 ridhi58

@ridhi58 I'll take a stub at this ASAP. Sorry for the inconvenience

musale avatar Jan 22 '25 08:01 musale

Hey @musale

Any update here?

ridhi58 avatar Jan 30 '25 16:01 ridhi58

Hello @ridhi58 thank you for your patience with this. I'm looking at it today.

musale avatar Feb 06 '25 06:02 musale