-
Notifications
You must be signed in to change notification settings - Fork 187
Add Org Link Expiration Policies #1038
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
still need descriptions
need add core/org stuff, just changed param set
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit d7ed980: ✅ Validation status: passed
For more details, please refer to the build report. |
|
waiting on pm review as well, please don't merge until @StephenRiceMSFT approves. |
|
|
||
| Specifies the recommended number of days before organization sharing links expire for this site. This setting provides a suggested expiration period to users when they create sharing links. | ||
|
|
||
| The value can be from 7 to 720 days and must be less than or equal to the maximum expiration value set by OrganizationSharingLinkMaxExpirationInDays. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| ### -OverrideTenantOrganizationLinkExpirationPolicy | ||
| {{ Fill OverrideTenantOrganizationLinkExpirationPolicy Description }} | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops :) #Closed
| ``` | ||
|
|
||
| ### -OrganizationSharingLinkMaxExpirationInDays | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing > Applicable: SharePoint Online for all the new params #Closed
| ``` | ||
|
|
||
| ### -CoreOrganizationSharingLinkMaxExpirationInDays | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing > Applicable: SharePoint Online for all the new params #Closed
|
|
||
| Specifies the recommended number of days before organization sharing links expire for all SharePoint sites (not including OneDrive sites). This setting provides a suggested expiration period to users when they create sharing links. | ||
|
|
||
| The value can be from 7 to 720 days and must be less than or equal to the maximum expiration value set by CoreOrganizationSharingLinkMaxExpirationInDays. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| Specifies the recommended number of days before organization sharing links expire for all OneDrive sites. This setting provides a suggested expiration period to users when they create sharing links. | ||
|
|
||
| The value can be from 7 to 720 days and must be less than or equal to the maximum expiration value set by OneDriveOrganizationSharingLinkMaxExpirationInDays. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit e874cdf: ✅ Validation status: passed
For more details, please refer to the build report. |
- Renamed OverrideTenantOrganizationLinkExpirationPolicy to OverrideTenantOrganizationSharingLinkExpirationPolicy in Set-SPOSite.md - Added proper description for OverrideTenantOrganizationSharingLinkExpirationPolicy parameter - Added '> Applicable: SharePoint Online' to all organization sharing link parameters in both Set-SPOSite.md and Set-SPOTenant.md - Added backticks around parameter cross-references per reviewer feedback - Standardized YAML type formatting to System.Int32 and System.Boolean
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 64ab74f: ✅ Validation status: passed
For more details, please refer to the build report. |
samkabue
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
![]()
No description provided.