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

Added new pack: Email Hippo #31167

Merged
merged 33 commits into from Dec 18, 2023
Merged

Added new pack: Email Hippo #31167

merged 33 commits into from Dec 18, 2023

Conversation

ilappe
Copy link
Contributor

@ilappe ilappe commented Nov 28, 2023

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-6673

Description

a new pack Email Hippo including:

  • Email Hippo integration

Must have

  • Tests
  • Documentation

Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/README.md Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/README.md Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/README.md Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/README.md Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/README.md Outdated Show resolved Hide resolved
@ShirleyDenkberg
Copy link
Contributor

@michalgold @DeanArbel @idovandijk Doc review completed.

ilappe and others added 5 commits November 28, 2023 17:33
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
@ShirleyDenkberg
Copy link
Contributor

@ilappe Don't forget to create content for the pack readme.

@ilappe
Copy link
Contributor Author

ilappe commented Nov 29, 2023

@ilappe Don't forget to create content for the pack readme.

Thanks @ShirleyDenkberg
i create it
can you please review it ?

@ShirleyDenkberg
Copy link
Contributor

@ilappe I reviewed the pack readme. One small edit.

ilappe and others added 4 commits November 29, 2023 15:58
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Copy link
Contributor

@DeanArbel DeanArbel left a comment

Choose a reason for hiding this comment

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

Impressive work, well done.
Let's do a demo.

Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml Outdated Show resolved Hide resolved
Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml Outdated Show resolved Hide resolved
@ilappe ilappe merged commit cedd926 into master Dec 18, 2023
18 of 19 checks passed
@ilappe ilappe deleted the ilappe/CIAC-6673/hippo_email branch December 18, 2023 13:37
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* new pack

* Apply suggestions from code review - part 1

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Apply suggestions from code review - part 2

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Apply suggestions from code review - part 3

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/EmailHippo/Integrations/EmailHippo/README.md

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/EmailHippo/Integrations/EmailHippo/README.md

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* A stricter test

* RN for CommonDashboards

* test playbook

* pack readme

* update pack_metadata

* Update Packs/EmailHippo/README.md

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* fix test fail

* compare dict

* Update Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml

Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>

* Update EmailHippo.yml

* Update EmailHippo.yml

* Revert "compare dict"

This reverts commit 13e8fbf55d5989d3dd526c6cf7a9f0e8fca62d91.

* update docker

* add no cover

* fix CR

* update the image

* fix pre commit

---------

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>
maimorag pushed a commit that referenced this pull request Dec 31, 2023
* new pack

* Apply suggestions from code review - part 1

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Apply suggestions from code review - part 2

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Apply suggestions from code review - part 3

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/EmailHippo/Integrations/EmailHippo/README.md

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/EmailHippo/Integrations/EmailHippo/README.md

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* A stricter test

* RN for CommonDashboards

* test playbook

* pack readme

* update pack_metadata

* Update Packs/EmailHippo/README.md

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* fix test fail

* compare dict

* Update Packs/EmailHippo/Integrations/EmailHippo/EmailHippo.yml

Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>

* Update EmailHippo.yml

* Update EmailHippo.yml

* Revert "compare dict"

This reverts commit 13e8fbf55d5989d3dd526c6cf7a9f0e8fca62d91.

* update docker

* add no cover

* fix CR

* update the image

* fix pre commit

---------

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants