fix: correct filter concatenation in mgt-people-picker (#2822)

microsoftgraph · Nov 2, 2023 · 18925a9 · 18925a9
1 parent 14e7a56
commit 18925a9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/mgt-components/src/graph/graph.people.ts b/packages/mgt-components/src/graph/graph.people.ts
@@ -135,7 +135,7 @@ export const findPeople = async (
 
   if (filters !== '') {
     // Adding the default people filters to the search filters
-    filter += `${filter} and ${filters}`;
+    filter += `${filters}`;
   }
   let graphResult: CollectionResponse<Person>;
   try {

diff --git a/stories/components/peoplePicker/peoplePicker.properties.stories.js b/stories/components/peoplePicker/peoplePicker.properties.stories.js
@@ -298,7 +298,11 @@ export const pickerUserFilters = () => html`
 
 export const pickerPeopleFilters = () => html`
   <mgt-people-picker
-      people-filters="jobTitle eq 'Web Marketing Manager'">
+      people-filters="jobTitle eq 'Retail Manager'">
+  </mgt-people-picker>
+
+  <mgt-people-picker
+    people-filters="personType/class eq 'Person' and personType/subclass eq 'OrganizationUser'">
   </mgt-people-picker>
 `;