Skip to content

CRTX-240216 - Enhancement - msft_azure_ad_raw #43786

Open
akshotiamit-pa wants to merge 3 commits intomasterfrom
CRTX-240216-Enhancement-msft_azure_ad_raw
Open

CRTX-240216 - Enhancement - msft_azure_ad_raw #43786
akshotiamit-pa wants to merge 3 commits intomasterfrom
CRTX-240216-Enhancement-msft_azure_ad_raw

Conversation

@akshotiamit-pa
Copy link
Copy Markdown
Contributor

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

Description

Must have

  • Tests
  • Documentation

@content-bot
Copy link
Copy Markdown
Contributor

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @marketplace-ai-reviewer start review - Initiate a full AI code review
  • @marketplace-ai-reviewer re-review - Incremental review for new commits

@akshotiamit-pa akshotiamit-pa force-pushed the CRTX-240216-Enhancement-msft_azure_ad_raw branch from 7935200 to 64c43c1 Compare April 9, 2026 10:44
@akshotiamit-pa akshotiamit-pa added bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. labels Apr 9, 2026
@marketplace-ai-reviewer marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Apr 9, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

Copy link
Copy Markdown
Contributor

@marketplace-ai-reviewer marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

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

Thanks for submitting your PR!
Before moving on with the detailed review, please take a moment to address the following general comments.

  • Your PR title is very broad and the description is empty. Please update the PR description to explicitly mention the specific enhancements being made (e.g., adding city and country location mappings, and including error codes in the outcome reason).

@richardbluestone richardbluestone requested review from julieschwartz18 and removed request for richardbluestone April 9, 2026 11:32
Copy link
Copy Markdown
Contributor

@julieschwartz18 julieschwartz18 left a comment

Choose a reason for hiding this comment

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

@akshotiamit-pa Doc review complete, minor formatting edits in the release notes. In the meantime adding the label docs-approved

@akshotiamit-pa akshotiamit-pa added the se-approved Pack available in SE label Apr 9, 2026
@akshotiamit-pa
Copy link
Copy Markdown
Contributor Author

@content-bot start review

@content-bot
Copy link
Copy Markdown
Contributor

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

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

Labels

bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines se-approved Pack available in SE

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants