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

Fix incorrect (ThrottleRateActionBytesPerSecond) parameter to correct one (ThrottleRateActionBitsPerSecond) in New-NetQosPolicy #3700

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Tamrin007
Copy link

@Tamrin007 Tamrin007 commented Nov 10, 2023

PR Summary

This change fixes the problem that the New-NetQosPolicy documentation contains an explanation and an example using an incorrect parameter.

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

ThrottleRateActionBitsPerSecond
@Tamrin007 Tamrin007 changed the title Fix wrong (ThrottleRateActionBytesPerSecond) parameter to right one (ThrottleRateActionBitsPerSecond) in New-NetQosPolicy Fix incorrect (ThrottleRateActionBytesPerSecond) parameter to correct one (ThrottleRateActionBitsPerSecond) in New-NetQosPolicy Nov 10, 2023
Copy link
Contributor

Learn Build status updates of commit 18df435:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2012-ps/netqos/New-NetQosPolicy.md ✅Succeeded View (winserver2012-ps)
docset/winserver2012r2-ps/netqos/New-NetQosPolicy.md ✅Succeeded View (winserver2012r2-ps)

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit b546a7a:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2012-ps/netqos/New-NetQosPolicy.md ✅Succeeded View (winserver2012-ps)
docset/winserver2012r2-ps/netqos/New-NetQosPolicy.md ✅Succeeded View (winserver2012r2-ps)

For more details, please refer to the build report.

For any questions, please:

@Tamrin007
Copy link
Author

Hi @get-itips @msbemba @scanum, thank you for taking the time to review and work on this PR. It looks like all reviews and checks have passed. Are there any additional actions required before this request can be merged?

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.

6 participants