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

Integrates the reporting plugin into the construction process #340

Conversation

yenienserrano
Copy link
Member

@yenienserrano yenienserrano commented Oct 3, 2024

Description

Adds the reporting plugin to the package build process

Issues Resolved

Screenshot

https://github.com/yenienserrano/wazuh-dashboard/actions/runs/11162798673

image

image

Test

  1. Run the package creation script with your and it should generate the package correctly using the package created from our reporting repository.

Check List

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

@yenienserrano yenienserrano self-assigned this Oct 3, 2024
@yenienserrano yenienserrano linked an issue Oct 3, 2024 that may be closed by this pull request
1 task
@guidomodarelli
Copy link

guidomodarelli commented Oct 3, 2024

Test 🟢
image

image

@Desvelao
Copy link
Member

Desvelao commented Oct 8, 2024

Test 🟢

Extracting the generated .tar.gz file from the Docker container, I got the following package.json of the reportingDashboards plugin:

console# cat plugins/reportsDashboards/package.json 
{
  "name": "reports-dashboards",
  "version": "2.16.0.0",
  "description": "OpenSearch Dashboards Reports Plugin",
  "license": "Apache-2.0",
  "main": "index.ts",
  "opensearchDashboards": {
    "version": "2.16.0",
    "templateVersion": "2.16.0"
  },
  "wazuh": {
    "version": "5.0.0",
    "revision": "00"
  },
  "scripts": {
    "osd": "node ../../scripts/osd",
    "opensearch": "node ../../scripts/opensearch",
    "lint:es": "node ../../scripts/eslint",
    "lint": "yarn lint:es",
    "start": "yarn plugin-helpers start",
    "build": "yarn plugin-helpers build",
    "test": "../../node_modules/.bin/jest --config ./test/jest.config.js",
    "cypress:run": "cypress run",
    "cypress:open": "cypress open",
    "plugin-helpers": "node ../../scripts/plugin_helpers",
    "postinstall": "node ./scripts/postinstall.js"
  },
  "dependencies": {
    "babel-polyfill": "^6.26.0",
    "cron-validator": "^1.1.1",
    "dompurify": "^3.0.11",
    "elastic-builder": "^2.7.1",
    "enzyme-adapter-react-16": "^1.15.5",
    "exceljs": "^4.4.0",
    "html2canvas": "1.4.1",
    "jest-fetch-mock": "^3.0.3",
    "jquery": "^3.5.0",
    "jsdom": "18.0.0",
    "json-2-csv": "^3.20.0",
    "jspdf": "^2.5.1",
    "react-addons-test-utils": "^15.6.2",
    "react-id-generator": "^3.0.1",
    "react-markdown": "^4.3.1",
    "react-mde": "^10.2.1",
    "react-native-base64": "^0.0.2",
    "react-native-i18n": "^2.0.15",
    "react-navigation": "^4.3.9",
    "react-router-dom": "^5.3.0",
    "react-toast-notifications": "^2.4.0",
    "set-interval-async": "1.0.33",
    "showdown": "^1.9.1",
    "tesseract.js": "^4.0.2"
  },
  "devDependencies": {
    "@elastic/eslint-import-resolver-kibana": "link:../../packages/osd-eslint-import-resolver-opensearch-dashboards",
    "@types/dompurify": "^3.0.5",
    "@types/enzyme-adapter-react-16": "^1.0.6",
    "@types/jsdom": "^16.2.3",
    "@types/react": "^16.14.23",
    "@types/react-addons-test-utils": "^0.14.25",
    "@types/react-dom": "^16.9.8",
    "@types/react-test-renderer": "^18.0.0",
    "@types/set-interval-async": "^1.0.0",
    "@types/showdown": "^1.9.3",
    "babel-jest": "^27.5.1",
    "cypress": "^13.6.0",
    "elastic-builder": "^2.7.1",
    "eslint-plugin-babel": "^5.3.1",
    "eslint-plugin-no-unsanitized": "^3.0.2",
    "eslint-plugin-prefer-object-spread": "^1.2.1",
    "identity-obj-proxy": "^3.0.0",
    "jest-dom": "^4.0.0",
    "react-test-renderer": "^16.12.0",
    "replace-in-file": "^6.3.5",
    "ts-jest": "^29.1.0"
  },
  "resolutions": {
    "trim": "^1.0.0",
    "doc-path": "3.1.0",
    "y18n": "^5.0.5",
    "lodash": "^4.17.21",
    "path-parse": "^1.0.7",
    "glob-parent": "^6.0.2",
    "css-what": "^5.0.1",
    "ansi-regex": "5.0.1",
    "json-schema": "0.4.0",
    "ws": "^7.5.10",
    "minimatch": "^3.0.5",
    "terser": "^4.8.1",
    "moment": "^2.29.4",
    "loader-utils": "^2.0.3",
    "yaml": "^2.2.2",
    "tough-cookie": "^4.1.3",
    "semver": "^7.5.2",
    "word-wrap": "^1.2.4",
    "@cypress/request": "^3.0.0",
    "@babel/traverse": "^7.23.2",
    "debug": "^4.3.4",
    "browserify-sign": "^4.2.2",
    "braces": "^3.0.3"
  }
}

CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Member

@Desvelao Desvelao left a comment

Choose a reason for hiding this comment

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

LGTM!

@Tostti Tostti merged commit 162c0ca into master Oct 9, 2024
33 checks passed
@Tostti Tostti deleted the enhancement/338-integrate-the-reporting-plugin-in-the-package-building-process branch October 9, 2024 15:12
yenienserrano added a commit that referenced this pull request Oct 28, 2024
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.

Integrate the reporting plugin in the package building process
4 participants