fix: show max while picking multiple groups (#3289)

Mnickii · web-flow · commit d2add09ae110 · 2024-09-11T13:43:36.000+03:00
diff --git a/packages/mgt-components/src/graph/graph.groups.ts b/packages/mgt-components/src/graph/graph.groups.ts
@@ -149,7 +149,8 @@ export const findGroups = async (
 
     filterQuery = filterQuery ? `${filterQuery} and ` : '';
     for (const filter of filterGroups) {
-      batch.get(filter, `/groups?$filter=${filterQuery + filter}`, validGroupQueryScopes);
+      const fullUrl = `/groups?$filter=${filterQuery + filter}&$top=${top}`;
+      batch.get(filter, fullUrl, validGroupQueryScopes);
     }
 
     try {
diff --git a/stories/components/peoplePicker/peoplePicker.properties.stories.js b/stories/components/peoplePicker/peoplePicker.properties.stories.js
@@ -187,6 +187,11 @@ export const pickMultipleGroups = () => html`
   <!-- group-type can be "any", "unified", "security", "mailenabledsecurity", "distribution" -->
 `;
 
+export const pickMultipleGroupsShowMax = () => html`
+  <mgt-people-picker type="group" group-type="unified,security,mailenabledsecurity" show-max="3"></mgt-people-picker>
+  <!-- group-type can be "any", "unified", "security", "mailenabledsecurity", "distribution" -->
+`;
+
 export const pickPeople = () => html`
   <mgt-people-picker type="person"></mgt-people-picker>
   <!-- type can be "any", "person", "group" -->

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,8 @@ export const findGroups = async (`
`149`	`149`
`150`	`150`	filterQuery = filterQuery ? `${filterQuery} and ` : '';
`151`	`151`	`for (const filter of filterGroups) {`
`152`		- batch.get(filter, `/groups?$filter=${filterQuery + filter}`, validGroupQueryScopes);
	`152`	+ const fullUrl = `/groups?$filter=${filterQuery + filter}&$top=${top}`;
	`153`	`+ batch.get(filter, fullUrl, validGroupQueryScopes);`
`153`	`154`	`}`
`154`	`155`
`155`	`156`	`try {`