Skip to content

Update security-incident-update.md - #10010

Open
Joseph-Cassidy wants to merge 1 commit into
microsoftgraph:mainfrom
Joseph-Cassidy:main
Open

Update security-incident-update.md#10010
Joseph-Cassidy wants to merge 1 commit into
microsoftgraph:mainfrom
Joseph-Cassidy:main

Conversation

@Joseph-Cassidy

Copy link
Copy Markdown

Changed confirmedUserActivity to confirmedActivity under determination as this appears to be breaking the update incident API request.

Changed confirmedUserActivity to confirmedActivity under determination
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit cad1778:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/security-incident-update.md ✅Succeeded

For more details, please refer to the build report.

@v-regandowner

Copy link
Copy Markdown

Danipocket - Can this minor change be merged without moving to the private repo?

Thanks!

@v-regandowner Regan Downer (v-regandowner) added the aq-pr-triaged PR has been processed by the Aquent PR review team label Aug 4, 2026
@JarbasHorst

Copy link
Copy Markdown
Contributor

Hi Regan Downer (@v-regandowner), hi Danipocket, we can merge this PR. However, it's fixing just part of the issue. I will create a follow-up PR to fix the other appearances of this enum member.

@JarbasHorst

Copy link
Copy Markdown
Contributor

Here the PR that completes this one: #10015

@David1997sb David1997sb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

hi Jarbas Horst (@JarbasHorst) we need to have all the checks in this PR before merging it

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

Labels

aq-pr-triaged PR has been processed by the Aquent PR review team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants