Skip to content

Conversation

@navba-MSFT
Copy link
Contributor

@navba-MSFT navba-MSFT commented May 17, 2022

…able parameter

On this page:
https://docs.microsoft.com/en-us/powershell/module/az.network/new-azapplicationgatewayfirewallexclusionconfig?view=azps-7.5.0

It seems it would be useful to document the valid values for the -Variable and -Operator parameters

Fixes #18106

Description

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:
    • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
      • {Please put the link here}
    • the markdown help files have been regenerated using the commands listed here

…able parameter

On this page:
https://docs.microsoft.com/en-us/powershell/module/az.network/new-azapplicationgatewayfirewallexclusionconfig?view=azps-7.5.0

It seems it would be useful to document the valid values for the -Variable and -Operator parameters

Fixes Azure#18106
@navba-MSFT
Copy link
Contributor Author

@MikhailTryakhov Could you please approve and merge this once you get a chance ? Thanks in advance.

@dingmeng-xue dingmeng-xue added Network - Application Gateway Service Attention This issue is responsible by Azure service team. labels May 20, 2022
@VeryEarly
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@VeryEarly VeryEarly self-assigned this May 23, 2022
@navba-MSFT
Copy link
Contributor Author

@VeryEarly Thanks for approving the request. Once the Tests succeeds, could you please merge this PR ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Doc - Conceptual Network - Application Gateway Service Attention This issue is responsible by Azure service team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Doc]: New-AzApplicationGatewayFirewallExclusionConfig

3 participants