Skip to content

GetRange #17511

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

Merged
merged 25 commits into from
Feb 20, 2022
Merged

GetRange #17511

merged 25 commits into from
Feb 20, 2022

Conversation

AradCarmi
Copy link
Contributor

@AradCarmi AradCarmi commented Feb 6, 2022

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

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: https://github.com/demisto/etc/issues/32930

Description

  • Created transformer to get range of values in an array

Screenshots

Paste here any images that will help the reviewer

Minimum version of Cortex XSOAR

  • 6.0.0
  • 6.1.0
  • 6.2.0
  • 6.5.0

Does it break backward compatibility?

  • Yes
  • No

Must have

  • Tests
  • Documentation

@AradCarmi AradCarmi changed the title First commit GetRange Feb 10, 2022
@AradCarmi AradCarmi requested a review from dantavori February 13, 2022 11:25
Copy link
Contributor

@dantavori dantavori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great! let's try it out tomorrow before the demo.

@AradCarmi AradCarmi requested a review from dantavori February 17, 2022 15:38
@content-bot
Copy link
Collaborator

@AradCarmi AradCarmi merged commit 2628eaa into master Feb 20, 2022
@AradCarmi AradCarmi deleted the Transformer_range branch February 20, 2022 09:33
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.

3 participants