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

Fix fetching incidents #34124

Merged
merged 1 commit into from
May 1, 2024
Merged

Conversation

content-bot
Copy link
Collaborator

Original External PR

external pull request

Contributor

@chkp-christiang

Status

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

Description

Fetch incidents method is raising an error since a parameter was not properly casted to integer.

Must have

  • Tests
  • Documentation

@content-bot content-bot added Contribution Thank you! Contributions are always welcome! Partner Contribution Form Filled Whether contribution form filled or not. Partner Support Level Indicates that the contribution is for Partner supported pack Internal PR Partner-Approved labels Apr 30, 2024
@content-bot content-bot requested a review from maimorag April 30, 2024 07:46
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CheckPointHEC/Integrations/CheckPointHEC
   CheckPointHEC.py2433486%193, 214, 220, 226, 232, 238, 244, 250, 252, 254, 256, 262, 268, 274, 280, 286, 288, 290, 292, 298, 304, 310, 319, 321, 376–377, 379, 389, 406, 411, 536, 549, 562, 575
TOTAL2433486% 

Tests Skipped Failures Errors Time
22 0 💤 0 ❌ 0 🔥 6.540s ⏱️

@maimorag maimorag merged commit 626c137 into master May 1, 2024
42 checks passed
@maimorag maimorag deleted the contrib/CheckPointSW_fix-fetch-incidents branch May 1, 2024 14:11
pal-xmco pushed a commit to pal-xmco/content that referenced this pull request Jun 19, 2024
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! docs-approved Internal PR Partner Support Level Indicates that the contribution is for Partner supported pack Partner Partner-Approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants