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

(#1162) Made changes to CCM client setup features #1163

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

imm0rtalsupp0rt
Copy link
Contributor

Description Of Changes

These changes address typos that created incorrect links within the Chocolatey Central Management client setup docs.

Motivation and Context

The changes made are to reflect feature settings and mirror the configuration settings further up the page.

Testing

  • I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

  • Minor documentation fix (typos etc.).
  • Major documentation change (refactoring, reformatting or adding documentation to existing page).
  • New documentation page added.
  • The change I have made should have a video added, and I have raised an issue for this.
    • Issue #

Change Checklist

  • Requires a change to menu structure (top or left-hand side)/
  • Menu structure has been updated

Related Issue

Fixes #1162

Copy link
Member

@pauby pauby left a comment

Choose a reason for hiding this comment

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

Rather than defining these values here again, can we not simply point to the config page https://docs.chocolatey.org/en-us/configuration/#chocolatey-central-management ? Alternatively, make it a snippet and include it in both places.

Both solutions mean we only have to ever update it once, in one place.

@imm0rtalsupp0rt
Copy link
Contributor Author

@pauby that's a larger scope of work, and since there is discussion of overhauling CCM documentation, I'd rather get this fixed now and then discuss with my team the best way forward for the entirety of 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.

Fix documentation error in client setup documentation for Chocolatey Central Management
2 participants