Skip to content

Conversation

@jamesaepp
Copy link
Contributor

Type

  • Bug Fix
  • New Feature
  • Sample

Related Issues?

Partially fixes #5135

What is in this Pull Request ?

Documentation improvement. I will note I clarified the type of application permission with (Delegated) and (Application). From a quick search of the existing code/MD files I didn't see an existing convention.

@jamesaepp
Copy link
Contributor Author

To ensure there's no confusion: I release my contribution to the care of the project maintainers and license it under the same MIT license as the project.

@jamesaepp
Copy link
Contributor Author

Clearly I'm still new to github issues/PR workflows. The commits in my fork now include a fix for another issue, #5140

@jackpoz
Copy link
Contributor

jackpoz commented Oct 27, 2025

I would recommend to create a new branch when you work on a fix, so you can work on multiple fixes at the same time in multiple branches.

@jamesaepp
Copy link
Contributor Author

Maintainers let me know if you want me to take any action to ensure this PR is just solving a single issue and branch everything out correctly.

These are just documentation updates, so stakes are low but I don't want to be a burden either.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances documentation for two cmdlets related to Microsoft Purview sensitivity labels by adding permission requirements and billing information.

  • Adds required permissions section to Get-PnPAvailableSensitivityLabel documentation
  • Adds billing requirements and related links for Add-PnPFileSensitivityLabel

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
documentation/Get-PnPAvailableSensitivityLabel.md Adds Required Permissions section documenting both delegated and application permission options
documentation/Add-PnPFileSensitivityLabel.md Adds Required Billing section explaining API billing requirements and related documentation links

@KoenZomers KoenZomers merged commit 6b33cff into pnp:dev Oct 31, 2025
4 checks passed
KoenZomers added a commit that referenced this pull request Oct 31, 2025
Added James Epp for his contribution through PR #5136
@KoenZomers
Copy link
Collaborator

Thanks @jamesaepp for your nice contribution to the documentation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Get-PnPAvailableSensitivityLabel: Forbidden (403):

3 participants