Skip to content

[7.x] [ILM] Revisit searchable snapshot field after new redesign (#90793)#90937

Merged
jloleysens merged 2 commits intoelastic:7.xfrom
jloleysens:backport/7.x/pr-90793
Feb 15, 2021
Merged

[7.x] [ILM] Revisit searchable snapshot field after new redesign (#90793)#90937
jloleysens merged 2 commits intoelastic:7.xfrom
jloleysens:backport/7.x/pr-90793

Conversation

@jloleysens
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…0793)

* moved searchable snapshot field out of cold phase accordian

* refactor styling to padding top and bottom to get advanced settings drop down to sit flush with side of panel

* Error clearing fix and cosmetic changes

- the error state of the form would not clear correctly if the
  erroring field was unmounted. The logic for clearing form errors
  was also incorrectly using "keys" instead of "values".
- updated the width of wait for snapshot policy field to be the
  same as other fields

* fix hook dependency causing clearError to be called

* slight improvement to component integration test

* re-add singleSelection to snapshot policiy field config

* refactored Phase component API and fixed typo in comment

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@jloleysens jloleysens added the backport This PR is a backport of another PR label Feb 10, 2021
@jloleysens
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibanamachine
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
indexLifecycleManagement 239.3KB 240.0KB +718.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jloleysens jloleysens merged commit ce9ca8e into elastic:7.x Feb 15, 2021
@jloleysens jloleysens deleted the backport/7.x/pr-90793 branch February 15, 2021 11:04
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.

2 participants