-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Dataset Quality] Migrate Dataset Quality Flyout to a Dedicated Page #184572
Labels
Comments
achyutjhunjhunwala
added
Team:obs-ux-logs
Observability Logs User Experience Team
Feature:Dataset Health
labels
May 31, 2024
Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs) |
@achyutjhunjhunwala is this a prereq for https://github.com/elastic/observability-dev/issues/3232? |
Yes |
Issue updated with the latest Figma link + latest screenshots |
This was referenced Jul 8, 2024
achyutjhunjhunwala
added a commit
that referenced
this issue
Aug 14, 2024
…ponent (#189532) ## Summary Relates to - #184572 Figma Design - https://www.figma.com/design/8WVWLeVn8mvoUm0VGgbSbB/Data-set-quality-V2?node-id=3564-73485&t=KADTdNFiiOBJ7rOS-0 **NOTE: This PR is part of a multi series PRs. Hence expect it to not do everything.** ### What are we going to do? 1. The content of the flyout, henceforth will be known as Dataset Quality Details is being copied to a component with the same name. 2. This component can be initialised as page, like in Management app or used as an individual component in a flyout in Unified Doc Viewer for example. As scope of this PR, a page in Management app has been created which will load this detailed component. A new route will be created with breadcrumb. 3. This page will co-live with the Flyout for now, accessible only via direct URL. In subsequent PR, when we remove the Flyout completely, we will change the action in Dataset Quality to instead of opening a Flyout to navigating to this new page. ### What's in this PR ? 1. As part of this change, i have created a complete new State Machine, Controller for Dataset Quality Details component which is responsible for replicating the Flyout. 2. A dedicated route registered under `/details` where this component will live at the moment. Sample URL will look like this `http://localhost:5601/pfd/app/management/data/data_quality/details?pageState=(dataStream:logs-synth.1-default,v:1)` 3. The individual components which currently load inside the flyout may be duplicated for time being. 4. Validation when no data stream provided. 5. Breadcrumb for the Management page ### What's not in this PR 1. Tests needs to be migrated, they will be done as part of the Next PR 2. Telemetry for Flyout has been removed. It will be added as part of next PR. 3. Existing Flyout code has not be removed. That needs to be removed and the old state machine needs to be meticulously cleaned. 6. Swapping the Click to Open Flyout to Page needs to be done when the above 3 are ready. ## Screenshot ### Good scenario <img width="1482" alt="image" src="https://github.com/user-attachments/assets/4409eb57-89d5-477c-a946-1b7a45df074c"> ### When datastream does not exist <img width="1527" alt="image" src="https://github.com/user-attachments/assets/66d735aa-8f0f-4fb8-b57c-4d22cecad2c7"> ### When invalid state is provided by the page, it redirects to parent Dataset Quality Page ![Aug-09-2024 13-55-54](https://github.com/user-attachments/assets/ea8379c5-0642-458c-8164-f50a17818895) --------- Co-authored-by: kibanamachine <[email protected]>
6 tasks
achyutjhunjhunwala
added a commit
that referenced
this issue
Aug 27, 2024
) ## Summary closes #184572 After the merge of the 1st [PR](#189532) around Flyout migration, this PR covers the remaining bits. - [x] Adding same telemetry to the page which was present in the flyout - [x] Create a Locator and use it in the Table to connect the main page with Details page - [x] Update locator in Unified Doc Viewer - [x] Migrate all kind of tests which were present for flyout to the page - [x] Remove everything which was once called Dataset Quality Flyout. - [x] Make build green - Yellow is also acceptable --------- Co-authored-by: kibanamachine <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
📓 Summary
The Dataset Quality currently offers a Flyout which displays quite some data about specific DataStream.
Goal of this ticket is to migrate this Flyout into a Dedicated Page. This would help us to add more features especially to another flyout for the Fiy It flow to the Degraded Fields.
Designs
Figma Link
Technical Tasks
Stack management > Data > Data Quality
Nice to have
dataset-quality
plugin, this will allow us to reuse the page in places likeDatastreams > Quality
orDocs flyout > Quality
The text was updated successfully, but these errors were encountered: