Skip to content

Conversation

olangness
Copy link
Contributor

Changing the cmdlet name as well as adding a descirption

@olangness
Copy link
Contributor Author

@samschan-ms Can you add yourself as a reviewer

Copy link

Learn Build status updates of commit e3d3685:

✅ Validation status: passed

File Status Preview URL Details
exchange/exchange-ps/exchange/Set-PolicyConfig.md ✅Succeeded View (exchange-ps)

For more details, please refer to the build report.

For any questions, please:

@chrisda chrisda self-assigned this Dec 8, 2023
@chrisda
Copy link
Contributor

chrisda commented Dec 8, 2023

@olangness, we shouldn't publish this until the change is available to ~50% WW minumum.

@olangness
Copy link
Contributor Author

@chrisda This has been available worldwide for over a year

@chrisda
Copy link
Contributor

chrisda commented Dec 8, 2023

@olangness, I checked in two different M365 E5 no SDF no flighting tenants (one really old; one really new) where the account is added to a custom role group with all available Email & collaboration roles assigned, and this is what I see in Security & Compliance PowerShell:

PS C:\Users\jdoe> (Get-Command Set-PolicyConfig).parameters.keys | Sort-Object
CaseHoldPolicyLimit
ClassificationScheme
ComplianceUrl
Confirm
Debug
DlpAppGroups
DlpAppGroupsPsws
DlpNetworkShareGroups
DlpPrinterGroups
DlpRemovableMediaGroups
DocumentIsUnsupportedSeverity
EnableAdvancedRuleBuilder
EnableLabelCoauth
EnableSpoAipMigration
EndpointDlpGlobalSettings
EndpointDlpGlobalSettingsPsws
ErrorAction
ErrorVariable
ExtendTeamsDlpPoliciesToSharePointOneDrive
Identity
InformationAction
InformationBarrierMode
InformationBarrierPeopleSearchRestriction
InformationVariable
IsDlpSimulationOptedIn
OnPremisesWorkload
OutBuffer
OutVariable
PipelineVariable
ProcessingLimitExceededSeverity
PurviewLabelConsent
ReservedForFutureUse
RetentionForwardCrawl
RuleErrorAction
SenderAddressLocation
SiteGroups
SiteGroupsPsws
Verbose
WarningAction
WarningVariable
WhatIf

@samschan-ms
Copy link
Contributor

samschan-ms commented Dec 8, 2023

ExtendTeamsDlpPoliciesToSharePointOneDrive

Yes @chrisda , doc update is for the description of ExtendTeamsDlpPoliciesToSharePointOneDrive. All othe ExtendTeams* parameters are internal Microsoft use. you only see it on Get-PolicyConfig. it's not available in Set-PolicyConfig.

@chrisda
Copy link
Contributor

chrisda commented Dec 9, 2023

@samschan-ms, I believe your statement, but this PR seeks to replace the name of the ExtendTeamsDlpPoliciesToSharePointOneDrive parameter that's currently available (whether it's supposed to be or not) with the name ExtendTeamsDlpToSpoOdbConsent, which isn't available to me in my ordinary E5 tenants.

If the intent is to state that ExtendTeamsDlpPoliciesToSharePointOneDrive doesn't function or shouldn't be available on Set-PolicyConfig, then this PR should seek to modify the parameter description (only) to say:

This parameter is reserved for internal Microsoft use.

@samschan-ms
Copy link
Contributor

samschan-ms commented Dec 11, 2023

@olangness can you modify this PR? remove the change in361? there is no need to update the parameter name. it should simply be an update to the description.

@chrisda
Copy link
Contributor

chrisda commented Dec 11, 2023

@samschan-ms, I took the liberty of modifying the parameter name as you described. Is it OK?

Copy link

Learn Build status updates of commit 4616bde:

✅ Validation status: passed

File Status Preview URL Details
exchange/exchange-ps/exchange/Set-PolicyConfig.md ✅Succeeded View (exchange-ps)

For more details, please refer to the build report.

For any questions, please:

@samschan-ms
Copy link
Contributor

thank you @chrisda

Copy link

Learn Build status updates of commit 183033c:

✅ Validation status: passed

File Status Preview URL Details
exchange/exchange-ps/exchange/Set-PolicyConfig.md ✅Succeeded View (exchange-ps)

For more details, please refer to the build report.

For any questions, please:

@chrisda chrisda merged commit 89669e9 into MicrosoftDocs:main Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants