Skip to content

[8.19] [ES|QL] fix: don't let clear ESQL control type (#226122)#226210

Merged
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-226122
Jul 2, 2025
Merged

[8.19] [ES|QL] fix: don't let clear ESQL control type (#226122)#226210
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-226122

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

## Summary
When attempting to clear the control type an uncaught error occurs.
```
shared_form_components.tsx:56 Uncaught TypeError: Cannot read properties of undefined (reading 'key')
    at shared_form_components.tsx:56:1
```

As the type is mandatory, make the select un-clearable.

<img width="581" alt="image"
src="https://github.com/user-attachments/assets/e7b10080-0886-4a03-b8eb-4b8cca597294"
/>

(cherry picked from commit 1b17e02)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 2, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) July 2, 2025 13:38
@kibanamachine kibanamachine merged commit 9426fac into elastic:8.19 Jul 2, 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
esql 254.7KB 254.8KB +15.0B

cc @sddonne

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