Skip to content

Comments

Console now autocompletes indexes#8557

Merged
kobelb merged 1 commit intoelastic:masterfrom
kobelb:console-index-autocomplete
Oct 12, 2016
Merged

Console now autocompletes indexes#8557
kobelb merged 1 commit intoelastic:masterfrom
kobelb:console-index-autocomplete

Conversation

@kobelb
Copy link
Contributor

@kobelb kobelb commented Oct 5, 2016

Closes #8531

The mappings.retrieveAutocompleteInfoFromServer function used to be called when the 'es' module called all callbacks registered via addServerChangeListener. This was removed by fee819d so I'm instead calling this method in the app.init function.

There might be a more appropriate place to put this call.

@kobelb kobelb added the review label Oct 6, 2016
@stacey-gammon stacey-gammon self-assigned this Oct 6, 2016
Copy link

@stacey-gammon stacey-gammon left a comment

Choose a reason for hiding this comment

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

tested locally and LGTM.

@epixa epixa changed the title Resolving bug with console not autocompleting indexes Console now autocompletes indexes Oct 8, 2016
@lukasolson lukasolson self-assigned this Oct 10, 2016
@lukasolson
Copy link
Contributor

LGTM

@kobelb kobelb merged commit 64227dc into elastic:master Oct 12, 2016
elastic-jasper added a commit that referenced this pull request Oct 12, 2016
---------

**Commit 1:**
Resolving bug with console not autocompleting indexes

* Original sha: 0ae3779
* Authored by = <brandon.kobel@elastic.co> on 2016-10-05T15:07:43Z
kobelb added a commit that referenced this pull request Oct 12, 2016
@kobelb
Copy link
Contributor Author

kobelb commented Oct 12, 2016

@epixa based on our conversations regarding adopting ES's policy of what gets into a patch version, is this something that you think should be merged into 5.0.0? Still trying to calibrate my definition of 'critical'.

@epixa
Copy link
Contributor

epixa commented Oct 12, 2016

@kobelb Nope

@kobelb kobelb deleted the console-index-autocomplete branch October 12, 2016 13:34
@epixa epixa added v5.1.1 and removed v5.1.0 labels Dec 8, 2016
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
---------

**Commit 1:**
Resolving bug with console not autocompleting indexes

* Original sha: ff54de62552b2f566e7423202f79028f0cadb421 [formerly 0ae3779]
* Authored by = <brandon.kobel@elastic.co> on 2016-10-05T15:07:43Z


Former-commit-id: c2b74fd
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants