Merge branch 'pnp:v2-dev' into v2-dev

Author: ravichandran-blog

CHANGELOG.JSON

@@ -7,15 +7,19 @@
           "`DynamicForm`: New Control: Dynamic form [#878](https://github.com/pnp/sp-dev-fx-controls-react/issues/878)"
         ],
         "enhancements": [
-          "`ComboBoxListItemPicker` and `ListItemPicker`: Add label to control [#910](https://github.com/pnp/sp-dev-fx-controls-react/pull/910)"
+          "`ComboBoxListItemPicker` and `ListItemPicker`: Add label to control [#910](https://github.com/pnp/sp-dev-fx-controls-react/pull/910)",
+          "`PeoplePicker`: new property `groupId`. [#917](https://github.com/pnp/sp-dev-fx-controls-react/pull/917)"
         ],
         "fixes": [
-          "`SitePicker`: `SitePicker` does not display initialsites until you click the dropdown to select [#895](https://github.com/pnp/sp-dev-fx-controls-react/issues/895)"
+          "`SitePicker`: `SitePicker` does not display initialsites until you click the dropdown to select [#895](https://github.com/pnp/sp-dev-fx-controls-react/issues/895)",
+          "`FilePicker`: invalid CSS: relative in quotes. [#930](https://github.com/pnp/sp-dev-fx-controls-react/pull/930)"
         ]
       },
       "contributions": [
-        "[Ravichandran Krishnasamy](https://github.com/ravichandran-blog)",
-        "[Peter Paul Kirschner](https://github.com/petkir)"
+        "[Nikolay Belykh](https://github.com/nbelyh)",
+        "[Patrik Hellgren](https://github.com/patrikhellgren)",
+        "[Peter Paul Kirschner](https://github.com/petkir)",
+        "[Ravichandran Krishnasamy](https://github.com/ravichandran-blog)"
       ]
     },
     {

docs/documentation/docs/controls/PeoplePicker.md

@@ -57,7 +57,8 @@ The People picker control can be configured with the following properties:
 | ---- | ---- | ---- | ---- | ---- |
 | context | WebPartContext | yes | Context of the current web part. | |
 | titleText | string | no | Text to be displayed on the control | |
-| groupName | string | no | group from which users are fetched. Leave it blank if need to filter all users | _none_ |
+| groupName | string | no | Group from which users are fetched. Leave it blank if need to filter all users. When both groupName and groupId specified groupName takes precedence. | _none_ |
+| groupId | number | no | Group from which users are fetched. Leave it blank if need to filter all users. When both groupId and groupName specified groupName takes precedence. | _none_ |
 | personSelectionLimit | number | no | Defines the limit of people that can be selected in the control | 1 |
 | required | boolean | no | Set if the control is required or not | false |
 | disabled | boolean | no | Set if the control is disabled or not | false |

src/controls/filePicker/FilePicker.module.scss

@@ -176,7 +176,7 @@
 }
 .scrollablePaneWrapper {
   height: 75vh;
-  position: 'relative';
+  position: relative;
   overflow: hidden;
 }
 

src/controls/peoplepicker/IPeoplePicker.ts

@@ -30,6 +30,10 @@ export interface IPeoplePickerProps {
    */
   groupName?: string;
   /**
+   * Id of SharePoint Group
+   */
+   groupId?: number;
+   /**
    * Maximum number of suggestions to show in the full suggestion list. (default: 5)
    */
   suggestionsLimit?: number;

src/controls/peoplepicker/PeoplePickerComponent.tsx

@@ -75,7 +75,7 @@ export class PeoplePicker extends React.Component<IPeoplePickerProps, IPeoplePic
    * Get initial persons
    */
   private async getInitialPersons(props: IPeoplePickerProps) {
-    const { groupName, webAbsoluteUrl, defaultSelectedUsers, ensureUser, principalTypes } = props;
+    const { groupName, groupId, webAbsoluteUrl, defaultSelectedUsers, ensureUser, principalTypes } = props;
     // Check if a group property was provided, and get the group ID
     if (groupName) {
       this.groupId = await this.peopleSearchService.getGroupId(groupName, webAbsoluteUrl);
@@ -85,6 +85,8 @@ export class PeoplePicker extends React.Component<IPeoplePickerProps, IPeoplePic
         });
         return;
       }
+    } else if (groupId) {
+      this.groupId = groupId;
     } else {
       this.groupId = null;
     }