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

[Discover] Always request unmapped fields in a single doc view and a context view #91825

Merged
merged 3 commits into from
Feb 18, 2021

Conversation

majagrubic
Copy link
Contributor

@majagrubic majagrubic commented Feb 18, 2021

Summary

Partially fixes: #91688

As a quick fix, we'll always request unmapped fields if using the fields API in the single document view and a single doc view.

Checklist

Delete any items that are not applicable to this PR.

- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] Documentation was added for features that require explanation or tutorials

For maintainers

@majagrubic majagrubic marked this pull request as ready for review February 18, 2021 12:44
@majagrubic majagrubic requested a review from a team February 18, 2021 12:44
@majagrubic majagrubic added v7.12.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Discover Discover Application labels Feb 18, 2021
@kertal kertal added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Feb 18, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

Copy link
Member

@kertal kertal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 , Request contains include_unmapped, unmapped fields are displayed
Bildschirmfoto 2021-02-18 um 14 09 52

@kertal kertal added the v8.0.0 label Feb 18, 2021
@majagrubic majagrubic changed the title [Discover] Always request unmapped fields in a single doc view [Discover] Always request unmapped fields in a single doc view and a context view Feb 18, 2021
Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected, context and single doc views are always showing unmapped fields (for fields api and source)

@kibanamachine
Copy link
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
discover 408.6KB 408.7KB +96.0B

History

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

@majagrubic majagrubic merged commit c5dabc2 into elastic:master Feb 18, 2021
@majagrubic majagrubic deleted the single_doc_view branch February 18, 2021 16:26
majagrubic pushed a commit to majagrubic/kibana that referenced this pull request Feb 18, 2021
…context view (elastic#91825)

* [Discover] Always request unmapped fields in a single doc view

* Updating unit test

* Request unmapped fields in context view
majagrubic pushed a commit to majagrubic/kibana that referenced this pull request Feb 18, 2021
…context view (elastic#91825)

* [Discover] Always request unmapped fields in a single doc view

* Updating unit test

* Request unmapped fields in context view
majagrubic pushed a commit that referenced this pull request Feb 19, 2021
…context view (#91825) (#91933)

* [Discover] Always request unmapped fields in a single doc view

* Updating unit test

* Request unmapped fields in context view
majagrubic pushed a commit that referenced this pull request Feb 19, 2021
…context view (#91825) (#91934)

* [Discover] Always request unmapped fields in a single doc view

* Updating unit test

* Request unmapped fields in context view
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Discover Discover Application release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Discover: "Show unmapped fields" flag does not carry over to single doc viewer and context viewer
5 participants