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

Complete CyberArk IAM/Secrets Manager Adoption #30257

Merged
merged 10 commits into from
Oct 22, 2023

Conversation

MMCrosby
Copy link
Contributor

Per CyberArk's request

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: link to the issue

Description

A few sentences describing the overall goals of the pull request's commits.

Must have

  • Tests
  • Documentation

Per CyberArk's request
@CLAassistant
Copy link

CLAassistant commented Oct 18, 2023

CLA assistant check
All committers have signed the CLA.

@kgal-pan kgal-pan changed the title Update README.md Complete CyberArk IAM/Secrets Manager Adoption Oct 18, 2023
@kgal-pan kgal-pan self-requested a review October 18, 2023 12:09
@omerKarkKatz omerKarkKatz requested review from omerKarkKatz and removed request for sapirshuker October 18, 2023 12:28
Copy link
Contributor

@omerKarkKatz omerKarkKatz left a comment

Choose a reason for hiding this comment

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

@kgal-pan @MMCrosby
Hey, looks good thank you for the contribution and congratulations for the adoption of your pack!
A few notes.
Please bump the version of the pack
In the release notes please write something in the form of:

  • Updated the pack from xsoar support to Cyberack partner support.

Also please add to the pack readme file -
Note: Support for this Pack moved to the partner on [MONTH], [DAY], [YEAR].

Please contact the partner directly via the support link on the right. [link or mail]

As mentioned here as part of the adoption process.
Thanks again @omerKarkKatz

added adoption readme preface
@kgal-pan kgal-pan merged commit 539dc40 into demisto:contrib/MMCrosby_patch-2 Oct 22, 2023
18 of 19 checks passed
omerKarkKatz pushed a commit that referenced this pull request Oct 22, 2023
* Update README.md

Per CyberArk's request

* modified pack metadata

* added author image

* update author image

* decreased size of author image

* added secret ignore

* rm conflict patch

* revert vectra ai readme

* added rn

added adoption readme preface

---------

Co-authored-by: Melody Crosby <[email protected]>
Co-authored-by: Kobbi Gal <[email protected]>
Co-authored-by: kgal-pan <[email protected]>
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* Update README.md

Per CyberArk's request

* modified pack metadata

* added author image

* update author image

* decreased size of author image

* added secret ignore

* rm conflict patch

* revert vectra ai readme

* added rn

added adoption readme preface

---------

Co-authored-by: Melody Crosby <[email protected]>
Co-authored-by: Kobbi Gal <[email protected]>
Co-authored-by: kgal-pan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! External PR Partner Partner-Approved Xsoar Support Level Indicates that the contribution is for XSOAR supported pack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants