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 typo in inputs description for pan-os-custom-block-rule #20188
Fix typo in inputs description for pan-os-custom-block-rule #20188
Conversation
Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @MosheEichler will know he can start review the proposed changes. |
@issolahma Thank you for your contribution! Nicely done, can you please update the release-notes? |
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.
Hi issolahma,
Thanks for contributing!
@@ -1604,15 +1604,15 @@ script: | |||
name: rulename | |||
required: true | |||
- auto: PREDEFINED | |||
description: Where to move the rule. Can be "before", "after", "top", or "bottom". If you specify "top" or "bottom", you need to supply the "dst" argument. | |||
description: Where to move the rule. Can be "before", "after", "top", or "bottom". If you specify "before" or "afer", you need to supply the "dst" argument. |
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.
description: Where to move the rule. Can be "before", "after", "top", or "bottom". If you specify "before" or "afer", you need to supply the "dst" argument. | |
description: Where to move the rule. Can be "before", "after", "top", or "bottom". If you specify "before" or "after", you need to supply the "dst" argument. |
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.
Hi issolahma,
Thanks for contributing!
- Please see my comment about one type
- Please add a release notes with running the demisto-sdk update-release-notes command.
Please let me know when you done.
Hi MosheEichler and GuyAfik, |
For the Reviewer: Successfully created a pipeline in Gitlab with url: https://code.pan.run/xsoar/content/-/pipelines/3322799 |
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.
Nice work, Thank you!
fc1e398
into
demisto:contrib/issolahma_Panorama-integration-typo
* Fix typo in inputs description for pan-os-custom-block-rule (#20188) * Fix typo in inputs description for pan-os-custom-block-rule * Fixed other same typo in description * fix typo * Add release note * Update Packs/PAN-OS/ReleaseNotes/1_14_12.md Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com> * conflicts Co-authored-by: issolahma <22448610+issolahma@users.noreply.github.com> Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com> Co-authored-by: meichlerpanw <meichler@paloaltonetworks.com>
Status
Description
There is a typo in the bloc "Panorama - Create custom block rule" for the "where" and "dst" inputs descriptions.
The "dst" input must be filled only if you choose "before" or "after" in the "where" input, but it says "top" or "bottom".
Screenshots
Does it break backward compatibility?