Skip to content

[7.x] Add error if filter index pattern is missing (#66979)#68050

Merged
lizozom merged 1 commit intoelastic:7.xfrom
lizozom:backport/7.x/pr-66979
Jun 3, 2020
Merged

[7.x] Add error if filter index pattern is missing (#66979)#68050
lizozom merged 1 commit intoelastic:7.xfrom
lizozom:backport/7.x/pr-66979

Conversation

@lizozom
Copy link
Copy Markdown
Contributor

@lizozom lizozom commented Jun 3, 2020

Backports the following commits to 7.x:

* add error if filter index pattern is gone

* docs change - why?

* Fix i18n

* Added a functional test for broken filters (field + index pattern)

* Clarify readme

* Moved readme

* New warning status

* Remove getAll

* git pull upstream master

* Fix translation files

* Fix merge

* added filterbar texts

* disabled correction

* Disable check in maps test until elastic#64861 is resolved

* Fix tests, warning state is not disabled.

* Adjust warning filter - ignore filters from "foreign" index pattern, that are still applicable

* Add an additional unrelaeted filter test

* Update src/plugins/data/public/ui/filter_bar/_global_filter_item.scss

Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>

* Update src/plugins/data/public/ui/filter_bar/_global_filter_item.scss

Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>

* Fixed test data

* Revert mapping

* Update data with missing test

* Update test to match data

* Code review

Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@lizozom lizozom added the backport This PR is a backport of another PR label Jun 3, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

@lizozom lizozom merged commit 8f1cdca into elastic:7.x Jun 3, 2020
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