-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security Solution][Case] Show the current connector name in case view #93018
Conversation
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
.find('[data-test-subj="has-data-to-push-button"]') | ||
.first() | ||
.text() | ||
.includes('My Connector 2') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the dumb question but how does the new name get set? Is it changed by a different test? Or in one of the mock implementations above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! It is set in useConnectorsMock.mockImplementation(() => ({ connectors: connectorsMock, loading: false }));
.
… ilm/rollup-v2-action * 'ilm/rollup-v2-action' of github.com:elastic/kibana: [Security Solution][Case][Bug] Only add rule object for alert comments (#92977) [Security Solution][Case] Show the current connector name in case view (#93018) [Security Solution] Remove unused mock data (#92357) Adds mapping to the signals for the indicator rules that were missing (#92928) skip flaky suite (#85208) Cleanup spaces plugin (#91976) Control round and decimal places in Gauge Visualization when using aggregate functions like average (#91293) Added alerting ui mock for jest test (#92604) Remove "beta" label from URL Drilldown as it is now GA (#92859)
Summary
If you create a case with a connector selected the name of the connector is being saved in the case's saved object. Specifically:
connector: {id: string; name: string; ...}
. If you update the connector, the name is not updated in the case's saved object because the updating is happening on the action's side and we do not have a way to update our information after an action's update. This PR fixes this issue as the name of the connector is taken from the up to date list of all current connectors.Fixes: #92607
Checklist
Delete any items that are not applicable to this PR.
For maintainers