Skip to content

[9.0] [ResponseOps][Slack] Allow configuring and accessing only slack web api or webhook (#237640)#238257

Merged
js-jankisalvi merged 2 commits into
elastic:9.0from
js-jankisalvi:backport/9.0/pr-237640
Oct 9, 2025
Merged

[9.0] [ResponseOps][Slack] Allow configuring and accessing only slack web api or webhook (#237640)#238257
js-jankisalvi merged 2 commits into
elastic:9.0from
js-jankisalvi:backport/9.0/pr-237640

Conversation

@js-jankisalvi
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

…pi or webhook (elastic#237640)

## Summary

Fixes elastic#233306
Fixes elastic#233177

This PR allows to access only slack web api or slack webhook connector
while creating or updating a rule/connector if only one of them is
configured in kibana.yml.
If both of them configured, then connector flyout only shows slack
connector with webhook and web api sub types.

- When `xpack.actions.enabledActionTypes: ['.slack_api'] `

https://github.com/user-attachments/assets/6dd0aab4-963b-41ba-ba44-81d42338e2fd

- When `xpack.actions.enabledActionTypes: ['.slack'] `

https://github.com/user-attachments/assets/47b147dc-5d94-4c3e-b0e2-5bc94e987f0f

- When `xpack.actions.enabledActionTypes: ['.slack', '.slack_api'] `

https://github.com/user-attachments/assets/7d33fda8-b373-43c8-a6e4-656135c2e3cb

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [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
- [ ] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 44e7331)

# Conflicts:
#	src/platform/packages/shared/kbn-alerts-ui-shared/src/common/types/action_types.ts
#	x-pack/platform/plugins/shared/stack_connectors/public/connector_types/slack/slack.tsx
#	x-pack/platform/plugins/shared/stack_connectors/public/connector_types/slack_api/slack_api.tsx
#	x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/action_connector_form/action_type_menu.test.tsx
#	x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/action_connector_form/create_connector_flyout/index.test.tsx
@js-jankisalvi js-jankisalvi added the backport This PR is a backport of another PR label Oct 9, 2025
@js-jankisalvi js-jankisalvi enabled auto-merge (squash) October 9, 2025 13:28
@js-jankisalvi js-jankisalvi requested a review from a team October 9, 2025 13:29
@js-jankisalvi js-jankisalvi self-assigned this Oct 9, 2025
@js-jankisalvi js-jankisalvi merged commit 77b8c71 into elastic:9.0 Oct 9, 2025
9 checks passed
@js-jankisalvi js-jankisalvi deleted the backport/9.0/pr-237640 branch October 9, 2025 17:34
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
stackConnectors 342 343 +1

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/alerts-ui-shared 288 289 +1
triggersActionsUi 571 572 +1
total +2

Async chunks

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

id before after diff
observability 1.2MB 1.2MB +27.0B
triggersActionsUi 1.6MB 1.6MB +490.0B
total +517.0B

Page load bundle

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

id before after diff
stackConnectors 60.5KB 60.6KB +84.0B
Unknown metric groups

API count

id before after diff
@kbn/alerts-ui-shared 304 305 +1
triggersActionsUi 597 598 +1
total +2

History

cc @js-jankisalvi

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants