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 ServiceNow fetch ticket command on notes without dates #26466

Merged
merged 14 commits into from
May 29, 2023

Conversation

MosheEichler
Copy link
Contributor

@MosheEichler MosheEichler commented May 11, 2023

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://jira-hq.paloaltonetworks.local/browse/XSUP-24209

Description

replaced the split function with the partition function to avoid failures on notes without dates.

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
    • Further details:
  • No

Must have

  • Tests
  • Documentation

@MosheEichler MosheEichler requested review from a team and Shellyber as code owners May 11, 2023 14:50
@MosheEichler MosheEichler requested review from gal-forer and removed request for a team May 11, 2023 14:50
@MosheEichler MosheEichler self-assigned this May 11, 2023
@MosheEichler MosheEichler marked this pull request as draft May 11, 2023 14:50
#### Integrations
##### ServiceNow v2
- Updated the Docker image to: *demisto/python3:3.10.11.58677*.
- Fixed an issue where the ***servicenow-get-ticket-notes*** command would fail on tickets without author's name.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Fixed an issue where the ***servicenow-get-ticket-notes*** command would fail on tickets without author's name.
- Fixed an issue where the ***servicenow-get-ticket-notes*** command would fail on tickets without an author's name.

@ShirleyDenkberg
Copy link
Contributor

@Shellyber @gal-forer Doc review completed.

# Conflicts:
#	Packs/ServiceNow/Integrations/ServiceNowv2/ServiceNowv2.yml
#	Packs/ServiceNow/ReleaseNotes/2_5_21.md
#	Packs/ServiceNow/pack_metadata.json

#### Integrations
##### ServiceNow v2
- Fixed an issue where the ***servicenow-get-ticket-notes*** command would fail on tickets with notes that contain two or more new lines.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Fixed an issue where the ***servicenow-get-ticket-notes*** command would fail on tickets with notes that contain two or more new lines.
Fixed an issue where the ***servicenow-get-ticket-notes*** command would fail on tickets with notes that contain two or more new lines.

@MosheEichler MosheEichler marked this pull request as ready for review May 24, 2023 10:26
@MosheEichler MosheEichler merged commit c6a8507 into master May 29, 2023
@MosheEichler MosheEichler deleted the service_now_fetch_ticket branch May 29, 2023 10:25
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