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

[8.18] [Security GenAI] "Select a Connector" popup does not show up after the user selects any connector and then cancels it from Endpoint Insights. (#208907) (#208969) #212911

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

e40pud
Copy link
Contributor

@e40pud e40pud commented Mar 3, 2025

…e user selects any connector and then cancels it from Endpoint Insights. (elastic#208907) (elastic#208969)

## Summary

BUG: elastic#208907

This PR fixes the issue where user cannot select a different connector
type after mistakenly selecting a wrong one.

### Steps to reproduce without required endpoint installation:

1. Make sure there are no connectors
2. Open "AI Assistant" on one of the security solution pages
3. Press "(+) Add connector" button
4. Connector type selection modal is visible
5. Select "Amazon Bedrock" type (or any other connector type)
6. Cancel the modal
7. Press "(+) Add connector" button again

**ISSUE**: previously selected connector type is being displayed and
there is no way to switch between types
**EXPECTED**: we should show connector type selection modal once
previous one was closed

### Issue recording

https://github.com/user-attachments/assets/48052bf1-4e00-43b7-a63e-f8a7969b9dbf

### Fixed state recording

https://github.com/user-attachments/assets/48be1cc4-0326-43a1-bd57-bb82fc1f19eb
(cherry picked from commit b28036a)
@e40pud e40pud merged commit 27db1c1 into elastic:8.18 Mar 3, 2025
11 checks passed
@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
securitySolution 9.1MB 9.1MB -8.0B

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants