Skip to content

Completed with the enhancement and added new commands.#45

Open
charan-metron wants to merge 25 commits into
masterfrom
silentpush-commands-enhancement
Open

Completed with the enhancement and added new commands.#45
charan-metron wants to merge 25 commits into
masterfrom
silentpush-commands-enhancement

Conversation

@charan-metron

Copy link
Copy Markdown
Collaborator

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

response = self._http_request(method="PUT", url=url, data=payload)

if isinstance(response, dict) and response.get("errors"):
return {"error": f"Failed to add new indicators: {response['errors']}"}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Invalid initial error message (copy paste)

response = self._http_request(method="GET", url=url, params=params)

if isinstance(response, dict) and response.get("errors"):
return {"error": f"Failed to add new indicators: {response['errors']}"}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Invalid initial error message (copy paste)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants