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

DenyLocalLogOn description updated as incorrect #11951

Closed
wants to merge 1 commit into from

Conversation

silvermarkg
Copy link
Contributor

Description

Updated DenyLocalLogOn description as it was incorrectly referring to DenyLogOnAsService

Why

DenyLocalLogOn description is incorrect which could lead users to misinterpret the setting.

Changes

Updated DenyLocalLogOn description

Updated DenyLocalLogOn description as it was incorrect, refering to DenyLogOnAsService
Copy link
Contributor

Learn Build status updates of commit 61c5dd4:

✅ Validation status: passed

File Status Preview URL Details
windows/client-management/mdm/policy-csp-userrights.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@aczechowski
Copy link
Member

Thanks @silvermarkg for the edit, although unfortunately we won't be able to accept it. Most of the CSP documentation is now automated from the DDF and ADMX source files. So even if we took this change now, it would reset in a few weeks when the process runs. For more information, see Contributing to the CSP reference articles.

I'm going to keep this open though until next week for @vinaypamnani-msft to review, because it's tingling the back of my brain. Like maybe there's a known issue with this policy or something. 🤔

@silvermarkg
Copy link
Contributor Author

No problem and thanks for the link. At least it's been raised for awareness 😊

@vinaypamnani-msft
Copy link
Contributor

Closing this PR. I'd already filed a bug for this internally, once it's resolved, the docs will automagically pick up the changes.

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.

3 participants