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

Add changes from Wazuh to the platform #14

Closed
wants to merge 4 commits into from
Closed

Conversation

AlexRuiz7
Copy link
Member

@AlexRuiz7 AlexRuiz7 commented Feb 3, 2023

Description

The following list of changes have been applied:

  • Changes in CSS.
  • Logos and icons replacements.
  • Replacement of some texts (OpenSearch with Wazuh, for example).
  • Changes in the properties of some fonts.
  • Replace default values for some platform's settings, as applicationTitle and useExpandedHeader.
  • Replace Node options (Unhandled promise error when unable to reach Wazuh API wazuh-dashboard-plugins#3887).
  • Replacement of help links with Wazuh's (titles and URLs).
  • Disabled the Overview plugin.
  • Manage the attempts of installation of the Wazuh plugin (needs further work, but will be done in an individual issue).

List of changes that could not be applied and reason:

  • Replace redirections to /home with /app/wazuh. Reason: the Wazuh plugin is not installed by default, This redirection leads to a 404: app not found view. We need to find a better way to do this change.
  • Changes to the documentation links and replacement of the OSD version with the Wazuh version. Reason: the version field in the package.json file cannot be changes, as it must match the version of the OpenSearch Indexer. A new field must be added to store the Wazuh version, and new code must be created to read this field during and use it during runtime. Will be done in a separate issue.
  • Remove "New to OpenSearch Dashboards" message with link to OpenSearch Dashboards sample data in Dashboard plugin. Reason: this component has several entry points and collateral effects that we need to study. We'll probably need to disable the route to this view and remove all links to it. Some other views might be affected. We'll perform this change in a separate issue.

Issues Resolved

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@AlexRuiz7 AlexRuiz7 mentioned this pull request Feb 3, 2023
8 tasks
@AlexRuiz7 AlexRuiz7 linked an issue Feb 8, 2023 that may be closed by this pull request
@AlexRuiz7 AlexRuiz7 self-assigned this Feb 8, 2023
@AlexRuiz7 AlexRuiz7 marked this pull request as ready for review February 8, 2023 12:08
@AlexRuiz7 AlexRuiz7 changed the base branch from 2.4 to 2.4.1-wzd-base February 13, 2023 11:49
@AlexRuiz7 AlexRuiz7 changed the base branch from 2.4.1-wzd-base to 2.4.1 February 13, 2023 18:03
@AlexRuiz7 AlexRuiz7 marked this pull request as draft February 13, 2023 18:21
@AlexRuiz7 AlexRuiz7 marked this pull request as ready for review February 13, 2023 18:21
@AlexRuiz7
Copy link
Member Author

Closing this pull request because I removed the Cypress workflow, and it cannot be created again unless a new pull request is opened. The workflow had cached data that caused the tests to fail.

@AlexRuiz7 AlexRuiz7 closed this Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change CSS styles and images
1 participant