Skip to content
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

[7.x] [Ingest Pipelines] Processor forms for processors K-S (#75638) #76762

Merged
merged 2 commits into from
Sep 8, 2020

Conversation

jloleysens
Copy link
Contributor

Backports the following commits to 7.x:

* added kv processor form

* added lowercase processor form

* Added pipeline processor form

* added remove processor form

* added rename processor form

* Add script processor form

Also:

- Also refactored name of defaultBoolToUndef to undefinedIfValue
  to make it more broadly applicable for deserializing when
  default values have been provided
- Refactor to pass in original processor options value so that
  a form can initialize based on those values

* Updated the existing set processor form

- also fixed a bug that would render a horizontal line even if
  the form has no fields of its own (like the drop processor form)

* Added set_security_user form

* Added split processor form

- also added comments to set_security_user processor

* added sort processor form

* fix: duplicate i18n identifier

* added field to kv processor form for trim value

* replaced all editor heights with a clearer indication of the source of common heights

* remove unused translations

* remove unused translation

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@jloleysens
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
ingestPipelines 426 +10 416

async chunks size

id value diff baseline
ingestPipelines 731.6KB +25.8KB 705.8KB

History

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

@jloleysens jloleysens merged commit 495c7f8 into elastic:7.x Sep 8, 2020
@jloleysens jloleysens deleted the backport/7.x/pr-75638 branch September 8, 2020 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants