-
Notifications
You must be signed in to change notification settings - Fork 532
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
Add AuditEnabled to EXOSharedMailbox #5748
base: Dev
Are you sure you want to change the base?
Add AuditEnabled to EXOSharedMailbox #5748
Conversation
-PropertySets is not a valid property for the Get-Mailbox cmdlet. What are we trying to accomplish with this PR? |
I mistakenly added -PropertySets All to Get-Mailbox as I confused it with Get-EXOMailbox. I am not very experienced with Powershell yet, so I am sorry for the inconvenience. Thanks for your understanding! |
@NikCharlebois LGTM. |
@@ -93,6 +97,7 @@ function Get-TargetResource | |||
-RecipientTypeDetails 'SharedMailbox' ` | |||
-ResultSize Unlimited ` | |||
-ErrorAction Stop | |||
-PropertySets All |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing '`'
@@ -101,6 +106,7 @@ function Get-TargetResource | |||
-RecipientTypeDetails 'SharedMailbox' ` | |||
-ResultSize Unlimited ` | |||
-ErrorAction Stop | |||
-PropertySets All |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing '`'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PropertySets is not a valid property of the Get-Mailbox cmdlet
CHANGELOG.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like an old version of the change log. Please refresh your branch with the latest changes.
Pull Request (PR) description
This PR adds AuditEnabled to EXOSharedMailbox
This Pull Request (PR) fixes the following issues
None
Task list
Entry should say what was changed and how that affects users (if applicable), and
reference the issue being resolved (if applicable).