Skip to content
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

Remove SkipPriorityCleanupConfirmation parameter - nonavailability in code #12479

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

joagarwal
Copy link
Contributor

Submitted via Learn Editor.

@joagarwal joagarwal marked this pull request as ready for review January 29, 2025 06:11
Copy link

Learn Build status updates of commit c3f68f9:

⚠️ Validation status: warnings

File Status Preview URL Details
exchange/docs-conceptual/index.yml ⚠️Warning View (exchange-ps) Details
exchange/exchange-ps/exchange/New-RetentionCompliancePolicy.md ✅Succeeded View (exchange-ps)

exchange/docs-conceptual/index.yml

  • Line 11, Column 18: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.subservice': 'subservice' is not valid with 'ms.service' value 'exchange-online'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link

Learn Build status updates of commit 39162ed:

⚠️ Validation status: warnings

File Status Preview URL Details
exchange/docs-conceptual/index.yml ⚠️Warning View (exchange-ps) Details
exchange/exchange-ps/exchange/New-RetentionCompliancePolicy.md ✅Succeeded View (exchange-ps)

exchange/docs-conceptual/index.yml

  • Line 11, Column 18: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.subservice': 'subservice' is not valid with 'ms.service' value 'exchange-online'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@chrisda
Copy link
Contributor

chrisda commented Jan 29, 2025

@joagarwal, are you saying that this parameter is going away, or that it's not supposed to be available? I just checked my plain M365 E5 tenant, and it's still available.

I'll check back in in a couple of weeks to see if it's gone, and then I'll remove it. Otherwise, need to leave it documented with the description:

This parameter is reserved for internal Microsoft use.

@joagarwal
Copy link
Contributor Author

joagarwal commented Jan 31, 2025

@chrisda - Yes a change is rolling out to hide this param as it is not currently used. We can wait for it to saturate to merge this PR.

cc: @rimshukla

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.

2 participants