Skip to content

[7.x] [ILM] Migrate Delete phase and name field to Form Lib (#82834)#82949

Merged
jloleysens merged 1 commit intoelastic:7.xfrom
jloleysens:backport/7.x/pr-82834
Nov 9, 2020
Merged

[7.x] [ILM] Migrate Delete phase and name field to Form Lib (#82834)#82949
jloleysens merged 1 commit intoelastic:7.xfrom
jloleysens:backport/7.x/pr-82834

Conversation

@jloleysens
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* remove use of legacy state system and legacy serialization

* remove legacy min_age input component and re-add missing import

* rename shared -> shared_fields for more clarity

* some more cleanup and fixing regressions on policy name for creating new policy from existing policy

* move extract policy static code to lib folder and remove "policies" dir from services

* fix jest tests and minor policy flyout inconsistency

* remove legacy helper

* fix client integration tests

* fix min for set index priority

* moved save policy function into edit policy section

* remove unused translations

* refactor form files to own edit_policy/form folder

* remove "fix errors" badge to fix UX - users can see errors in line before pressing save so the value of this badge has diminished

* fix i18n after removing phase error badge

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 Nov 9, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
indexLifecycleManagement 112 110 -2

async chunks size

id before after diff
indexLifecycleManagement 224.5KB 210.7KB -13.8KB

page load bundle size

id before after diff
indexLifecycleManagement 77.0KB 76.8KB -188.0B

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

@jloleysens jloleysens merged commit f7dc92d into elastic:7.x Nov 9, 2020
@jloleysens jloleysens deleted the backport/7.x/pr-82834 branch November 9, 2020 16:10
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