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

Upgrade documentation according to the new interface look #7280

Merged
merged 117 commits into from
May 31, 2024

Conversation

Tostti
Copy link
Member

@Tostti Tostti commented May 2, 2024

Description

From version 4.8.0, the interface of Wazuh Dashboard had a big change.
This PR aims to replace all the images to match the new interface, as well as reviewing the references to changed sections.

Checks

Docs building

  • Compiles without warnings.

Code formatting and web optimization

  • Uses three spaces indentation.
  • Adds or updates meta descriptions accordingly.
  • Updates the redirects.js script if necessary (check this guide).

Writing style

  • Uses present tense, active voice, and semi-formal registry.
  • Uses short, simple sentences.
  • Uses bold for user interface elements, italics for key terms or emphasis, and code font for Bash commands, file names, REST paths, and code.

@asteriscos asteriscos changed the title Upgrade documentation according to the new inteface look Upgrade documentation according to the new interface look May 2, 2024
@Tostti Tostti marked this pull request as draft May 2, 2024 19:48
source/_static/js/redirects.js Outdated Show resolved Hide resolved
source/_static/js/redirects.js Outdated Show resolved Hide resolved
source/_static/js/redirects.js Show resolved Hide resolved
source/user-manual/ruleset/mitre.rst Outdated Show resolved Hide resolved
source/images/compliance/nist/alert-of-rule-id-550.png Outdated Show resolved Hide resolved
source/images/compliance/nist/expand-alert.png Outdated Show resolved Hide resolved
source/images/compliance/nist/the-full-log-field.png Outdated Show resolved Hide resolved
source/images/poc/fim-alerts-windows.png Outdated Show resolved Hide resolved
source/images/sca/macos-sca-check-results.png Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

The image doesn't show the Save role mapping button. It would be a plus since it's to be clicked in next step and it was visible in previous image. This happens to the similar images that follow. These images appear cropped and look incomplete to my eyes at least.

Copy link
Member

Choose a reason for hiding this comment

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

Improvement suggestion: Higher used memory value and lower available memory value could make the graph standout. Currently the used memory in the graph is little visible.

Copy link
Member

Choose a reason for hiding this comment

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

Can we display the X-axis form in the right as well? Can we use a different time range to add variety to the heat map? Currently we just show two tactics and two techniques only, unlike the previous image.

Copy link
Member

Choose a reason for hiding this comment

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

If we could show the Y-axis form on the right as well, it would be a plus.

Copy link
Member

Choose a reason for hiding this comment

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

Similar to one of the previous comments: A higher memory usage relative to the available memory would improve the memory size metric graph here showing two areas. Currently, yellow area is little visible.

source/images/wodles-oscap/pci-oscap.png Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

Text is about rule ID 92154 and MITRE ID T1053.005

Copy link
Member

@javimed javimed May 28, 2024

Choose a reason for hiding this comment

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

The PoC is specific to Ubuntu 22 and it's about pulling the NGINX image, running the container, stopping and removing the container. Some alerts are missing (and some others are not relevant).

Copy link
Member

Choose a reason for hiding this comment

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

Step 3 is "Ensure the Ubuntu endpoint is selected". The previous image showed how to ensure the Ubuntu was selected but this is not present in the new image.

Copy link
Member

Choose a reason for hiding this comment

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

Question: Is it ok to see the syscheck.event field empty for rule ID 550 alert while we see it with value modified in the previous image?

Copy link
Member

Choose a reason for hiding this comment

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

Previous image with

  • Data on the Evolution panel
  • masOS endpoint
    looked better than this one. The getting started guide is aimed mainly for new users so the nicer the better if possible.

Copy link
Member

Choose a reason for hiding this comment

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

This image is cropped at the bottom and graphs look empty

Copy link
Member

Choose a reason for hiding this comment

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

An additional image showing the specific configuration for the master node like in the previous image could be useful.

Copy link
Member

Choose a reason for hiding this comment

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

Line #77 says Stack management > ...

Copy link
Member

Choose a reason for hiding this comment

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

Missing last hyphen. The index pattern name must be my-custom-alerts-*.

Copy link
Member

Choose a reason for hiding this comment

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

macOS use case uses 10005 check instead

Copy link
Member

Choose a reason for hiding this comment

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

full_log value doesn't match with document text custom rule definition.

Copy link
Member

Choose a reason for hiding this comment

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

Document text says Download CSV in

To download this report, click on the **Download CSV** beneath the specific property you are interested in on the **Inventory data** page of the endpoint.

@javimed javimed merged commit 80f9000 into 4.8.0 May 31, 2024
3 checks passed
@javimed javimed deleted the upgrade-documentation-new-interface-look branch May 31, 2024 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level/task Task issue type/change Change requested
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Change the default theme in Wazuh Dashboard advanced settings
4 participants