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

ews search and compliance: return info file entry #11832

Merged
merged 1 commit into from
Mar 25, 2021

Conversation

glicht
Copy link
Contributor

@glicht glicht commented Mar 16, 2021

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.

Screenshots

Paste here any images that will help the reviewer

Minimum version of Demisto

  • 5.0.0
  • 5.5.0
  • 6.0.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests
  • Documentation

@glicht
Copy link
Contributor Author

glicht commented Mar 17, 2021

@amshamah419 please take a look at release notes


#### Integrations
##### O365 - Security And Compliance - Content Search (beta)
- Fixed the commands `o365-sc-get-search` and `o365-sc-get-search-action` to return a `InfoFile` entry (instead of a `File` entry) when the `export` argument is specified.
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 the commands `o365-sc-get-search` and `o365-sc-get-search-action` to return a `InfoFile` entry (instead of a `File` entry) when the `export` argument is specified.
- Fixed the ***o365-sc-get-search*** and ***o365-sc-get-search-action*** commands to return an `InfoFile` entry, as opposed to a `File` entry, when the `export` argument is specified.

@glicht glicht merged commit 860dd7f into master Mar 25, 2021
@glicht glicht deleted the pwsh-compliance-search-fix branch March 25, 2021 07:40
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