-
Notifications
You must be signed in to change notification settings - Fork 185
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
Remove embedded discover
#6120
Labels
Comments
6 tasks
I will close this issue, because the angularJS routing will be solved in the next stage of the release. |
Added pull request to the list of fixes: #6950 |
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
We want to remove the embedded discover and replace it with a more modern component. This will allow us to remove angular as a dependency, improve usability and performance. Also, this will allow us to adapt to the upcoming changes from the upstream OpenSearch related to the new
discover
, the changes of the OUI library and others.We use discover in multiple places, the most apparent one is the Events tabs:
But its components are also used to build other visualizations, managing filters and some events. The removal of the
discover
will require a refactor of these uses.Our strategy will be to create our own component based on the one we already created for the new inventory tab:
Screen.Recording.2023-10-25.at.10.52.34.mov
We will add to the current inventory tab the time filters and the timeline visualization as shown in the
discover
.In general our implementation strategy will be to add the necessary changes to make the
discover
redundant, and then, remove it. Also, we don't want to lose functionality, if there are use cases not identified in this issue, discuss them in the comments sections below. We will update the issue body with the conclusion.Functional requirements
As a user, I want to be able to:
Explore data:
Generate reports:
Explore event context:
Non-functional requirements
Implementation restrictions
Plan
Related fixes/features
The text was updated successfully, but these errors were encountered: