-
Notifications
You must be signed in to change notification settings - Fork 130
[Security] Visual event analyzer updates #2606
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
base: main
Are you sure you want to change the base?
Conversation
🔍 Preview links for changed docs |
When you first select a process, it appears in a loading state. If loading data for a given process fails, click **Reload `{{process-name}}`** beneath the process to reload the data. | ||
% Is this still valid? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @PhilippeOberti, is this statement still true? In the test environment I used, there was no loading state – the process details would appear immediately.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much more maintainable - thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few suggestions you can take or leave, otherwise LGTM
Co-authored-by: Mike Birnstiehl <[email protected]>
Resolves #2339 by updating the Visual event analyzer docs to reflect the latest UI. This PR also reduces the number of screenshots on the page, for easier maintenance.
Preview: Visual event analyzer