Skip to content

Commit 57a45cf

Browse files
authored
Merge pull request #4488 from NishkalankBezawada/feature/AddWarnings-to-PowerAutomate-Commandlets
Added RequiredPermissionWarnings to 4 different commandlets
2 parents bcb4de6 + a806899 commit 57a45cf

File tree

8 files changed

+50
-11
lines changed

8 files changed

+50
-11
lines changed

documentation/Get-PnPSiteVersionPolicy.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,12 @@ title: Get-PnPSiteVersionPolicy
1212
## SYNOPSIS
1313
Get version policy setting of the site.
1414

15+
**Required Permissions**
16+
17+
| Type | API/ Permission Name | Admin consent required |
18+
| --------------- | --------------------------------------- | -------- |
19+
| Delegated | AllSites.FullControl | yes |
20+
1521
## SYNTAX
1622

1723
```powershell

documentation/Set-PnPSiteVersionPolicy.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,13 @@ online version: https://pnp.github.io/powershell/cmdlets/Set-PnPSiteVersionPolic
1212
## SYNOPSIS
1313
Sets file version policy related properties on the site.
1414

15+
**Required Permissions**
16+
17+
| Type | API/ Permission Name | Admin consent required |
18+
| --------------- | --------------------------------------- | -------- |
19+
| Delegated | AllSites.FullControl | yes |
20+
21+
1522
## SYNTAX
1623

1724
```powershell

src/Commands/Lists/GetListItem.cs

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,23 @@
44
using System.Text;
55
using System.Xml.Linq;
66
using Microsoft.SharePoint.Client;
7-
7+
using PnP.PowerShell.Commands.Attributes;
88
using PnP.PowerShell.Commands.Base.PipeBinds;
99

1010
namespace PnP.PowerShell.Commands.Lists
1111
{
1212
[Cmdlet(VerbsCommon.Get, "PnPListItem", DefaultParameterSetName = ParameterSet_ALLITEMS)]
1313
[OutputType(typeof(ListItem))]
14+
[RequiredApiApplicationPermissions("sharepoint/Sites.Selected")]
15+
[RequiredApiApplicationPermissions("sharepoint/Sites.Read.All")]
16+
[RequiredApiApplicationPermissions("sharepoint/Sites.ReadWrite.All")]
17+
[RequiredApiApplicationPermissions("sharepoint/Sites.Manage.All")]
18+
[RequiredApiApplicationPermissions("sharepoint/Sites.FullControl.All")]
19+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.Read")]
20+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.Write")]
21+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.Manage")]
22+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.FullControl")]
23+
1424
public class GetListItem : PnPWebRetrievalsCmdlet<ListItem>
1525
{
1626
private const string ParameterSet_BYID = "By Id";

src/Commands/PowerPlatform/PowerAutomate/GetDeletedFlow.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,12 @@
44
using System.Linq;
55
using System.Management.Automation;
66
using PnP.PowerShell.Commands.Utilities;
7+
using PnP.PowerShell.Commands.Attributes;
78

89
namespace PnP.PowerShell.Commands.PowerPlatform.PowerAutomate
910
{
1011
[Cmdlet(VerbsCommon.Get, "PnPDeletedFlow", DefaultParameterSetName = ParameterSet_ALL)]
12+
[RequiredApiDelegatedPermissions("azure/user_impersonation")]
1113
public class GetDeletedFlow : PnPAzureManagementApiCmdlet
1214
{
1315
private const string ParameterSet_ALL = "All";

src/Commands/Purview/GetAvailableSensitivityLabel.cs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
using PnP.PowerShell.Commands.Base;
1+
using PnP.PowerShell.Commands.Attributes;
2+
using PnP.PowerShell.Commands.Base;
23
using PnP.PowerShell.Commands.Base.PipeBinds;
34
using PnP.PowerShell.Commands.Utilities.REST;
45
using System;
@@ -10,6 +11,8 @@ namespace PnP.PowerShell.Commands.Purview
1011
[Cmdlet(VerbsCommon.Get, "PnPAvailableSensitivityLabel")]
1112
[OutputType(typeof(IEnumerable<Model.Graph.Purview.InformationProtectionLabel>))]
1213
[OutputType(typeof(Model.Graph.Purview.InformationProtectionLabel))]
14+
[RequiredApiApplicationPermissions("graph/InformationProtectionPolicy.ReadAll")]
15+
[RequiredApiDelegatedPermissions("graph/InformationProtectionPolicy.Read")]
1316
public class GetAvailableSensitivityLabel : PnPGraphCmdlet
1417
{
1518
[Parameter(Mandatory = false)]

src/Commands/Site/GetSite.cs

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,22 @@
1-
using Microsoft.SharePoint.Client;
1+
using Microsoft.SharePoint.Client;
2+
using PnP.PowerShell.Commands.Attributes;
3+
using System;
4+
using System.Linq.Expressions;
25
using System.Management.Automation;
36

47
namespace PnP.PowerShell.Commands.Site
58
{
69
[Cmdlet(VerbsCommon.Get, "PnPSite")]
710
[OutputType(typeof(Microsoft.SharePoint.Client.Site))]
11+
[RequiredApiApplicationPermissions("sharepoint/Sites.Selected")]
12+
[RequiredApiApplicationPermissions("sharepoint/Sites.Read.All")]
13+
[RequiredApiApplicationPermissions("sharepoint/Sites.ReadWrite.All")]
14+
[RequiredApiApplicationPermissions("sharepoint/Sites.Manage.All")]
15+
[RequiredApiApplicationPermissions("sharepoint/Sites.FullControl.All")]
16+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.Read")]
17+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.Write")]
18+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.Manage")]
19+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.FullControl")]
820
public class GetSite : PnPRetrievalsCmdlet<Microsoft.SharePoint.Client.Site>
921
{
1022
protected override void ExecuteCmdlet()

src/Commands/Site/GetSiteVersionPolicy.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
using Microsoft.SharePoint.Client;
2-
3-
using System;
4-
using System.Linq.Expressions;
52
using System.Management.Automation;
63
using PnP.PowerShell.Commands.Model.SharePoint;
4+
using PnP.PowerShell.Commands.Attributes;
75

86
namespace PnP.PowerShell.Commands.Site
97
{
108
[Cmdlet(VerbsCommon.Get, "PnPSiteVersionPolicy")]
11-
[OutputType(typeof(PnP.PowerShell.Commands.Model.SharePoint.SiteVersionPolicy))]
9+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.FullControl")]
10+
[RequiredApiApplicationPermissions("sharepoint/Sites.FullControl.All")]
11+
[OutputType(typeof(SiteVersionPolicy))]
1212
public class GetSiteVersionPolicy : PnPSharePointCmdlet
1313
{
1414
protected override void ExecuteCmdlet()

src/Commands/Site/SetSiteVersionPolicy.cs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
using Microsoft.SharePoint.Client;
2-
3-
using System;
4-
using System.Linq.Expressions;
52
using System.Management.Automation;
6-
using PnP.PowerShell.Commands.Model.SharePoint;
3+
using PnP.PowerShell.Commands.Attributes;
74

85
namespace PnP.PowerShell.Commands.Site
96
{
107
[Cmdlet(VerbsCommon.Set, "PnPSiteVersionPolicy")]
8+
[RequiredApiDelegatedPermissions("sharepoint/AllSites.FullControl")]
9+
[RequiredApiApplicationPermissions("sharepoint/Sites.FullControl.All")]
1110
[OutputType(typeof(void))]
1211
public class SetSiteVersionPolicy : PnPSharePointCmdlet
1312
{

0 commit comments

Comments
 (0)