Skip to content

Comments

Add an example of uiSettings override#135385

Merged
matschaffer merged 5 commits intomainfrom
matschaffer-uisetting-override
Jun 30, 2022
Merged

Add an example of uiSettings override#135385
matschaffer merged 5 commits intomainfrom
matschaffer-uisetting-override

Conversation

@matschaffer
Copy link
Contributor

Summary

I couldn't find an email of uiSettings override, so I added one. See #135030 (comment) for context.

@matschaffer matschaffer added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting DevDocs Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// labels Jun 29, 2022
@matschaffer matschaffer requested a review from kertal June 29, 2022 00:55
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

Co-authored-by: Matthias Wilhelm <matthias.wilhelm@elastic.co>
Copy link
Contributor

@lukeelmers lukeelmers left a comment

Choose a reason for hiding this comment

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

LGTM! Couple of minor nits/ideas, take 'em or leave 'em.

matschaffer and others added 2 commits June 30, 2022 09:57
Co-authored-by: Luke Elmers <luke.elmers@elastic.co>
Co-authored-by: Luke Elmers <luke.elmers@elastic.co>
@matschaffer
Copy link
Contributor Author

@lukeelmers 's recommended setting works nice too. Merging!

@matschaffer matschaffer merged commit 8e392a6 into main Jun 30, 2022
@matschaffer matschaffer deleted the matschaffer-uisetting-override branch June 30, 2022 03:08
@matschaffer matschaffer self-assigned this Jun 30, 2022
yakhinvadim pushed a commit to yakhinvadim/kibana that referenced this pull request Jul 5, 2022
Co-authored-by: Matthias Wilhelm <matthias.wilhelm@elastic.co>
Co-authored-by: Luke Elmers <luke.elmers@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting DevDocs release_note:skip Skip the PR/issue when compiling release notes Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// v8.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants