-
Notifications
You must be signed in to change notification settings - Fork 603
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 Add-KdsRootKey.md #2951
Update Add-KdsRootKey.md #2951
Conversation
Modified the EffectiveTime parameter example, to include a day larger than 12. This should prevent confusion over the format to use (recommended as the American mm/dd/yyyy format, however any format should work as the parameter takes a DateTime, not a string)
Docs Build status updates of commit e70e043: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
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.
@Natfan Can you check and make the same edit in the other OS versions of the same cmdlet article? Thanks for your contribution.
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.
I'm wondering if it would be useful to re-write this example to be generic, i.e., Add-KdsRootKey -EffectiveTime (Get-Date).AddDays(12)
and adding Get-Date
to Related Links.
Regardless, this works, too. Thanks!
@tiburd |
Learn Build status updates of commit a328a12: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Modified the EffectiveTime parameter example, to include a day larger than 12. This should prevent confusion over the format to use (recommended as the American mm/dd/yyyy format, however any format should work as the parameter takes a DateTime, not a string)