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

Sort Index Dropdown and Implement ComboBox #342

Conversation

frankschlehofer
Copy link

@frankschlehofer frankschlehofer commented Oct 31, 2023

Description

This change sorts the index dropdown. It also uses a ComboBox to add search functionality.
Screenshot 2023-10-31 at 7 52 21 PM

Screenshot 2023-10-31 at 7 52 39 PM

Issues Resolved

Closes #293

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Frank Schlehofer added 6 commits October 24, 2023 18:57
Signed-off-by: Frank Schlehofer <franklschlehofer@gmail.com>
Signed-off-by: Frank Schlehofer <franklschlehofer@gmail.com>
Signed-off-by: Frank Schlehofer <franklschlehofer@gmail.com>
…nal onChangeSelectedIndex

Signed-off-by: Frank Schlehofer <franklschlehofer@gmail.com>
Implements the ComboBox that allows search functionality for the indexes. This also sorts the documentsIndexes array.
Signed-off-by: Frank Schlehofer <franklschlehofer@gmail.com>
@ashwin-pc
Copy link
Member

ashwin-pc commented Dec 20, 2023

@frankschlehofer can you use the DataSourceSelectable component from the data plugin in OSD for this? it should do the heavy lifting of showing you all the available datasources.

Its used in the Data Explorer plugin like this: https://github.com/opensearch-project/OpenSearch-Dashboards/blob/main/src/plugins/data_explorer/public/components/sidebar/index.tsx#L96-L103

Any future updates to how data sources are handled will be taken care of here :)

@sejli
Copy link
Member

sejli commented Dec 27, 2023

Going to close this now, since we have more work being done on #358.

@sejli sejli closed this Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEATURE] Sort Index Dropdown
3 participants