Skip to content

[9.0] [Search]: Fix Number type field to have correct property (#210462)#210692

Merged
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-210462
Feb 11, 2025
Merged

[9.0] [Search]: Fix Number type field to have correct property (#210462)#210692
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-210462

Conversation

@kibanamachine
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

)

## Summary

This PR includes the following updates:
- Converted `readOnly` fields to `disabled` to ensure consistency across
all view-only fields.
- Added an `isPreconfigured` check to disable the `num_allocations`
field. While `num_allocations` is the only updatable field for
`Elasticsearch` services, it cannot be modified if the endpoint is
preconfigured.

### Not preconfigured
![Screenshot 2025-02-10 at 3 54
32 PM](https://github.com/user-attachments/assets/a70aa6a4-69b5-4f75-95ba-f3793b6992d2)

### Preconfigured
![Screenshot 2025-02-10 at 3 54
19 PM](https://github.com/user-attachments/assets/4e174b33-307a-4707-974e-a91e1ca56649)

### 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

(cherry picked from commit dc0ccc4)
@kibanamachine kibanamachine merged commit 3167121 into elastic:9.0 Feb 11, 2025
4 checks passed
@elasticmachine
Copy link
Copy Markdown
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
indexManagement 765.0KB 765.2KB +149.0B
searchInferenceEndpoints 113.5KB 113.6KB +149.0B
stackConnectors 688.8KB 688.9KB +129.0B
total +427.0B

cc @Samiul-TheSoccerFan

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