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

Update Set-CMCollectionCloudSync.md #290

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

Conversation

rnicolson
Copy link

Changed parameter title from AADGroupName to AddGroupName. This is correct in all the other locations but the parameter title is wrong.

Thanks for contributing!

Use the template below to describe your pull request (PR) for Configuration Manager PowerShell docs. You may delete any unneeded information from the template.

PR status

If you're not ready for review, remove the X or deselect the checkbox below. The markdown for a checkbox is space sensitive [X] or [ ].

  • This PR is complete and ready for review.

Description of changes

Give your PR a brief title and describe your doc changes in the comments. Include why the changes are needed. Example:

**PR Title:** Fix Set-CMCollectionCloudSync parameter title

1. For Set-CMCollectionCloudSync:
   - Changed parameter title from AADGroupName to AddGroupName. This is correct in all the other locations but the parameter title is wrong

More information

Here are some resources that might be helpful while contributing:

Changed parameter title from AADGroupName to AddGroupName. This is correct in all the other locations but the parameter title is wrong.
@prmerger-automator
Copy link

@rnicolson : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Banreet
Copy link
Contributor

Banreet commented Sep 18, 2023

#sign-off

@rnicolson
Copy link
Author

I am new to making changes here. How does this end up merging. The ready-to-merge tag was added a while ago but this does not seem to be merged yet. Is there still something I need to do?

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