Skip to content

Commit

Permalink
Fix bug in help for New-AzureRmPolicyDefinition cmdlet
Browse files Browse the repository at this point in the history
  • Loading branch information
@mentat9
mentat9 committed Jan 11, 2019
1 parent 535a087 commit 2bf0e39
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/ResourceManager/Resources/Commands.Resources/ChangeLog.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
-->
## Current Release
* Minor changes for upcoming AzureRM to Az transition
* Correct documentation for New-AzureRmPolicyDefinition -Mode default value

## Version 6.7.2
* Update help documentation for resource identity related parameters in `New-AzureRmPolicyAssignment` and `Set-AzureRmPolicyAssignment`
Expand Down
2 changes: 1 addition & 1 deletion ...esourceManager/Resources/Commands.Resources/help/New-AzureRmPolicyDefinition.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ Aliases:

Required: False
Position: Named
Default value: None
Default value: All

This comment has been minimized.

Sign in to view

Copy link
@robga

robga Feb 24, 2021

I know our UI is using All as default.
Our backend is using Indexed as default. (We also added that to the swagger https://github.com/Azure/azure-rest-api-specs/blob/33ffbed9469628215ec6cd842699446d324fda94/specification/resources/resource-manager/Microsoft.Authorization/stable/2020-09-01/policyDefinitions.json#L520)
I think it's better to use indexed as default value here will be better.
Accept pipeline input: True (ByPropertyName)
Accept wildcard characters: False
```
Expand Down

0 comments on commit 2bf0e39

Please sign in to comment.