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

[Maps] clear transient layer when required fields are cleared #31726

Merged
merged 3 commits into from
Feb 22, 2019

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Feb 21, 2019

fixes #31725

ES Documents and ES Grid aggregation source allow for clearing index pattern and geo field. This PR ensures the temp layer is removed if the user clears either index pattern or geo field so the Add layer button is disabled in this state

Prevents users from getting into the below state

screen shot 2019-02-21 at 11 19 09 am

@nreese nreese added v7.0.0 [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v6.7.0 v7.2.0 labels Feb 21, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

💯 for picking this up. Nice it was caught in time

@@ -22,12 +22,17 @@ export class ESSearchSource extends AbstractESSource {
static description = 'Geospatial data from a Kibana index pattern';

static renderEditor({ onPreviewSource, inspectorAdapters }) {
const onSelect = (layerConfig) => {
const layerSource = new ESSearchSource({
Copy link
Contributor

Choose a reason for hiding this comment

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

🙇‍♂️ for rename

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@nreese nreese merged commit 0b14bb4 into elastic:master Feb 22, 2019
nreese added a commit to nreese/kibana that referenced this pull request Feb 22, 2019
…c#31726)

* [Maps] clear transient layer when required fields are cleared

* review feedback
nreese added a commit that referenced this pull request Feb 22, 2019
#31776)

* [Maps] clear transient layer when required fields are cleared

* review feedback
@nreese nreese added non-issue Indicates to automation that a pull request should not appear in the release notes release_note:skip Skip the PR/issue when compiling release notes and removed non-issue Indicates to automation that a pull request should not appear in the release notes labels Mar 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation release_note:skip Skip the PR/issue when compiling release notes v6.7.0 v7.0.0 v7.2.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User shouldn't be able to clear the values for Geospatial field under grid aggregation
3 participants