-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Explicit index pattern namespace #57194
Explicit index pattern namespace #57194
Conversation
…index-pattern-namespace
f45936c
to
1d1f3ae
Compare
…index-pattern-namespace
...egacy/core_plugins/kibana/public/discover/np_ready/components/field_chooser/field_chooser.js
Outdated
Show resolved
Hide resolved
…ata/index-pattern-namespace
…ata/index-pattern-namespace
@elasticmachine merge upstream |
…:lizozom/kibana into newplatform/data/index-pattern-namespace
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maps changes LGTM
code review
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code changes in the data plugin lgtm!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
App code changes LGTM. Tested discover and graph locally, all looks good.
* Explicit index pattern namespace * typing of index patterns * fix ts error on master * export FieldList * Move isFilterable into indexPatterns namespace * Server index pattern cleanup * comment * isDefault * karma mock fix + discover fix Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
* Explicit index pattern namespace * typing of index patterns * fix ts error on master * export FieldList * Move isFilterable into indexPatterns namespace * Server index pattern cleanup * comment * isDefault * karma mock fix + discover fix Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Summary
Part of #56881
This PR applies the proposed resolution for #52374, trying to balance code readability, API discoverability, DX and documentation generation using ApiExtractor on index patterns.
Checklist
Delete any items that are not applicable to this PR.
For maintainers