Skip to content

[Cases] Separate sync alert and auto-extract updates in activity#236519

Merged
christineweng merged 3 commits intoelastic:mainfrom
christineweng:cases-observables-settings-action
Oct 14, 2025
Merged

[Cases] Separate sync alert and auto-extract updates in activity#236519
christineweng merged 3 commits intoelastic:mainfrom
christineweng:cases-observables-settings-action

Conversation

@christineweng
Copy link
Contributor

@christineweng christineweng commented Sep 25, 2025

Summary

Ref: #236493

When sync alert or auto-extract toggle was turned on/off, both settings are mentioned in the activity log. This PR added additional logic in user action builder to compare before/after values, and sends the correct payload to user action.

Before
image

After
image

Checklist

@christineweng christineweng self-assigned this Sep 25, 2025
@christineweng christineweng added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:Cases Security Solution Cases team v9.2.0 labels Sep 25, 2025
@christineweng christineweng force-pushed the cases-observables-settings-action branch from eaaacf5 to 47753a1 Compare October 10, 2025 17:40
@christineweng christineweng added release_note:fix backport:version Backport to applied version labels v9.3.0 and removed release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Oct 10, 2025
@christineweng christineweng marked this pull request as ready for review October 10, 2025 17:51
@christineweng christineweng requested a review from a team as a code owner October 10, 2025 17:51
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cases (Team:Cases)

);

// Join labels with "and" if multiple, or return single label
return labels.length > 1 ? labels.join(' and ') : labels[0] || '';
Copy link
Contributor

Choose a reason for hiding this comment

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

Concatenating these labels with and will break translations. Imagine each label is translated to another language (e.g. Spanish) and then they're concatenated with and.
I would recommend, adding another set of labels here, or concatenating with e.g. , or &

Copy link
Contributor

Choose a reason for hiding this comment

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

+1 for the ,

@christineweng christineweng enabled auto-merge (squash) October 14, 2025 15:23
@christineweng christineweng merged commit fca6120 into elastic:main Oct 14, 2025
12 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.2

https://github.com/elastic/kibana/actions/runs/18503400740

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cases 1.4MB 1.4MB +150.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
cases 141.1KB 141.2KB +101.0B

History

cc @christineweng

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 14, 2025
…stic#236519)

## Summary

Ref: elastic#236493

When sync alert or auto-extract toggle was turned on/off, both settings
are mentioned in the activity log. This PR added additional logic in
user action builder to compare before/after values, and sends the
correct payload to user action.

Before
<img width="715" height="185" alt="image"
src="https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7"
/>

After
<img width="565" height="192" alt="image"
src="https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e"
/>

### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

(cherry picked from commit fca6120)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
9.2

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 14, 2025
#236519) (#238976)

# Backport

This will backport the following commits from `main` to `9.2`:
- [[Cases] Separate sync alert and auto-extract updates in activity
(#236519)](#236519)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"christineweng","email":"18648970+christineweng@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-10-14T16:30:11Z","message":"[Cases]
Separate sync alert and auto-extract updates in activity (#236519)\n\n##
Summary\n\nRef: https://github.com/elastic/kibana/issues/236493\n\nWhen
sync alert or auto-extract toggle was turned on/off, both settings\nare
mentioned in the activity log. This PR added additional logic in\nuser
action builder to compare before/after values, and sends the\ncorrect
payload to user action.\n\nBefore\n<img width=\"715\" height=\"185\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7\"\n/>\n\nAfter\n<img
width=\"565\" height=\"192\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e\"\n/>\n\n\n\n###
Checklist\n\n- [x] Any text added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common scenarios\n- [x] The PR
description includes the appropriate Release Notes section,\nand the
correct `release_note:*` label is applied per
the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n-
[x] Review the
[backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand
apply applicable `backport:*`
labels.","sha":"fca612083454b5f784805485b019f52c8bc81161","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","backport:version","Team:Cases","v9.2.0","v9.3.0"],"title":"[Cases]
Separate sync alert and auto-extract updates in
activity","number":236519,"url":"https://github.com/elastic/kibana/pull/236519","mergeCommit":{"message":"[Cases]
Separate sync alert and auto-extract updates in activity (#236519)\n\n##
Summary\n\nRef: https://github.com/elastic/kibana/issues/236493\n\nWhen
sync alert or auto-extract toggle was turned on/off, both settings\nare
mentioned in the activity log. This PR added additional logic in\nuser
action builder to compare before/after values, and sends the\ncorrect
payload to user action.\n\nBefore\n<img width=\"715\" height=\"185\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7\"\n/>\n\nAfter\n<img
width=\"565\" height=\"192\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e\"\n/>\n\n\n\n###
Checklist\n\n- [x] Any text added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common scenarios\n- [x] The PR
description includes the appropriate Release Notes section,\nand the
correct `release_note:*` label is applied per
the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n-
[x] Review the
[backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand
apply applicable `backport:*`
labels.","sha":"fca612083454b5f784805485b019f52c8bc81161"}},"sourceBranch":"main","suggestedTargetBranches":["9.2"],"targetPullRequestStates":[{"branch":"9.2","label":"v9.2.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/236519","number":236519,"mergeCommit":{"message":"[Cases]
Separate sync alert and auto-extract updates in activity (#236519)\n\n##
Summary\n\nRef: https://github.com/elastic/kibana/issues/236493\n\nWhen
sync alert or auto-extract toggle was turned on/off, both settings\nare
mentioned in the activity log. This PR added additional logic in\nuser
action builder to compare before/after values, and sends the\ncorrect
payload to user action.\n\nBefore\n<img width=\"715\" height=\"185\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7\"\n/>\n\nAfter\n<img
width=\"565\" height=\"192\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e\"\n/>\n\n\n\n###
Checklist\n\n- [x] Any text added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common scenarios\n- [x] The PR
description includes the appropriate Release Notes section,\nand the
correct `release_note:*` label is applied per
the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n-
[x] Review the
[backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand
apply applicable `backport:*`
labels.","sha":"fca612083454b5f784805485b019f52c8bc81161"}}]}]
BACKPORT-->

Co-authored-by: christineweng <18648970+christineweng@users.noreply.github.com>
ersin-erdal pushed a commit to ersin-erdal/kibana that referenced this pull request Oct 15, 2025
…stic#236519)

## Summary

Ref: elastic#236493

When sync alert or auto-extract toggle was turned on/off, both settings
are mentioned in the activity log. This PR added additional logic in
user action builder to compare before/after values, and sends the
correct payload to user action.

Before
<img width="715" height="185" alt="image"
src="https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7"
/>

After
<img width="565" height="192" alt="image"
src="https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e"
/>



### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
mgadewoll pushed a commit to tkajtoch/kibana that referenced this pull request Oct 17, 2025
…stic#236519)

## Summary

Ref: elastic#236493

When sync alert or auto-extract toggle was turned on/off, both settings
are mentioned in the activity log. This PR added additional logic in
user action builder to compare before/after values, and sends the
correct payload to user action.

Before
<img width="715" height="185" alt="image"
src="https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7"
/>

After
<img width="565" height="192" alt="image"
src="https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e"
/>



### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
rylnd pushed a commit to rylnd/kibana that referenced this pull request Oct 17, 2025
…stic#236519)

## Summary

Ref: elastic#236493

When sync alert or auto-extract toggle was turned on/off, both settings
are mentioned in the activity log. This PR added additional logic in
user action builder to compare before/after values, and sends the
correct payload to user action.

Before
<img width="715" height="185" alt="image"
src="https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7"
/>

After
<img width="565" height="192" alt="image"
src="https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e"
/>



### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
nickpeihl pushed a commit to nickpeihl/kibana that referenced this pull request Oct 23, 2025
…stic#236519)

## Summary

Ref: elastic#236493

When sync alert or auto-extract toggle was turned on/off, both settings
are mentioned in the activity log. This PR added additional logic in
user action builder to compare before/after values, and sends the
correct payload to user action.

Before
<img width="715" height="185" alt="image"
src="https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7"
/>

After
<img width="565" height="192" alt="image"
src="https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e"
/>



### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Oct 27, 2025
…stic#236519)

## Summary

Ref: elastic#236493

When sync alert or auto-extract toggle was turned on/off, both settings
are mentioned in the activity log. This PR added additional logic in
user action builder to compare before/after values, and sends the
correct payload to user action.

Before
<img width="715" height="185" alt="image"
src="https://github.com/user-attachments/assets/37dc0b5f-ab7b-4163-b00d-d6e74f4f6af7"
/>

After
<img width="565" height="192" alt="image"
src="https://github.com/user-attachments/assets/7d7aff13-78bc-4f93-8676-0d7baa13cf3e"
/>



### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:fix Team:Cases Security Solution Cases team v9.2.0 v9.3.0

5 participants