Conversation
hyaffe839
added
docs-approved
ready-for-pipeline-running
🤖 AI-Powered Code Review AvailableYou can leverage AI-powered code review to assist with this PR! Available Commands:
|
Coverage Report
|
||||||||||||||||||||||||||||||
marketplace-ai-reviewer
removed
the
ready-for-ai-review
|
🤖 Analysis started. Please wait for results... |
🤖 AI Review DisclaimerThis review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause. |
marketplace-ai-reviewer
left a comment
marketplace-ai-reviewer
left a comment
There was a problem hiding this comment.
Thanks for your contribution! I've left a few notes to help polish the PR. The main issue is a bug in the SearchIssues script where list inputs for issue_id are incorrectly parsed as strings, which breaks the digit validation; using argToList will easily fix this. Also, please make sure to fix the pack name header and uncomment the relevant lines in the release notes. Let me know if you have any questions!
@dantavori please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
dantavori
left a comment
dantavori
left a comment
There was a problem hiding this comment.
nice, please add unittests
|
Validate summary Verdict: PR can be force merged from validate perspective? ✅ |
4 participants
Contributing to Cortex XSOAR Content
Make sure to register your contribution by filling the contribution registration form
The Pull Request will be reviewed only after the contribution registration form is filled.
Status
Related Issues
Fixes: https://jira-dc.paloaltonetworks.com/browse/CRTX-239797
Description
Fixed an issue in SearchIssuesAgentix Script where providing multiple values to the issue_id argument caused them to be treated as invalid and removed, due to the comma-separated list not being recognized as numeric.
Must have