From 5a615c5e938f420305c77cc5052da8a63a0a2c0f Mon Sep 17 00:00:00 2001 From: mkholjur Date: Mon, 20 Feb 2023 12:56:18 +0100 Subject: [PATCH 1/2] Revert "chore: do not wait advisories to display most impactfull statistics (#946)" This reverts commit c9dac6d3919658d9e99625d016d012a314c3535d. --- src/App.scss | 5 +++++ .../StatusReports/AdvisoriesStatusReport.js | 4 ++-- src/SmartComponents/Advisories/Advisories.js | 2 +- src/SmartComponents/Advisories/Advisories.test.js | 3 +-- src/Utilities.scss | 3 +++ 5 files changed, 12 insertions(+), 5 deletions(-) create mode 100644 src/Utilities.scss diff --git a/src/App.scss b/src/App.scss index f568b435f..04b0f0a37 100644 --- a/src/App.scss +++ b/src/App.scss @@ -1,3 +1,8 @@ +// Importing Global Variables +@import "~@redhat-cloud-services/frontend-components-utilities/styles/_all"; +@import "~@redhat-cloud-services/frontend-components/index.css"; +@import "~@redhat-cloud-services/frontend-components-notifications/index.css"; + table.patchCompactInventory { .ins-composed-col { font-size: var(--pf-c-table--m-compact--FontSize); diff --git a/src/PresentationalComponents/StatusReports/AdvisoriesStatusReport.js b/src/PresentationalComponents/StatusReports/AdvisoriesStatusReport.js index b406d63ce..38b9c9864 100644 --- a/src/PresentationalComponents/StatusReports/AdvisoriesStatusReport.js +++ b/src/PresentationalComponents/StatusReports/AdvisoriesStatusReport.js @@ -82,14 +82,14 @@ const StatusCard = ({ advisory: { attributes, id } }) => ); const AdvisoriesStatusBar = () => { - const [advisories, setAdvisories] = React.useState({}); + const [advisories, setAdvisories] = React.useState([]); React.useEffect(async () => { setAdvisories( await fetchApplicableAdvisoriesApi({ limit: 4, sort: '-advisory_type_name,-applicable_systems' }) ); }, []); - return advisories.data && advisories.data.length && ( + return advisories.data && advisories.data.length && (
diff --git a/src/SmartComponents/Advisories/Advisories.js b/src/SmartComponents/Advisories/Advisories.js index e677e45be..8009022bc 100644 --- a/src/SmartComponents/Advisories/Advisories.js +++ b/src/SmartComponents/Advisories/Advisories.js @@ -125,7 +125,7 @@ const Advisories = ({ history }) => { return ( <React.Fragment> <Header title={intl.formatMessage(messages.titlesPatchAdvisories)} headerOUIA={'advisories'} /> - <AdvisoriesStatusReport/> + {metadata?.has_systems === true && <AdvisoriesStatusReport/>} <Main> <TableView columns={advisoriesColumns} diff --git a/src/SmartComponents/Advisories/Advisories.test.js b/src/SmartComponents/Advisories/Advisories.test.js index 508ff3059..e93ae6546 100644 --- a/src/SmartComponents/Advisories/Advisories.test.js +++ b/src/SmartComponents/Advisories/Advisories.test.js @@ -29,8 +29,7 @@ jest.mock('../../Utilities/api', () => ({ exportAdvisoriesCSV: jest.fn(() => Promise.resolve({ success: true }).catch((err) => console.log(err))), fetchSystems: jest.fn(() => Promise.resolve({ data: { id: 'testId' } }).catch((err) => console.log(err))), fetchViewAdvisoriesSystems: jest.fn(() => Promise.resolve({ success: true }).catch((err) => console.log(err))), - fetchIDs: jest.fn(() => Promise.resolve({ ids: [] }).catch((err) => console.log(err))), - fetchApplicableAdvisoriesApi: jest.fn(() => Promise.resolve({ success: true }).catch((err) => console.log(err))) + fetchIDs: jest.fn(() => Promise.resolve({ ids: [] }).catch((err) => console.log(err))) })); jest.mock('../../Utilities/constants', () => ({ diff --git a/src/Utilities.scss b/src/Utilities.scss new file mode 100644 index 000000000..45aaae589 --- /dev/null +++ b/src/Utilities.scss @@ -0,0 +1,3 @@ +// Sass Imports + +@import "~@redhat-cloud-services/frontend-components-utilities/styles/_all"; From 8b556ba427ef6e2a71916e25a63f24b95dccc4fb Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 1 Mar 2023 18:00:17 +0000 Subject: [PATCH 2/2] chore(deps): bump @patternfly/react-icons from 4.93.3 to 4.93.6 Bumps [@patternfly/react-icons](https://github.com/patternfly/patternfly-react) from 4.93.3 to 4.93.6. - [Release notes](https://github.com/patternfly/patternfly-react/releases) - [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/react-icons@4.93.3...@patternfly/react-icons@4.93.6) --- updated-dependencies: - dependency-name: "@patternfly/react-icons" dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> --- package-lock.json | 14 +++++++------- package.json | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/package-lock.json b/package-lock.json index b1ba90d38..359cee4ad 100644 --- a/package-lock.json +++ b/package-lock.json @@ -11,7 +11,7 @@ "@data-driven-forms/pf4-component-mapper": "^3.11.3", "@data-driven-forms/react-form-renderer": "^3.16.5", "@patternfly/react-core": "^4.267.6", - "@patternfly/react-icons": "^4.93.3", + "@patternfly/react-icons": "^4.93.6", "@patternfly/react-table": "^4.112.6", "@redhat-cloud-services/frontend-components": "3.9.27", "@redhat-cloud-services/frontend-components-notifications": "^3.2.12", @@ -3819,9 +3819,9 @@ } }, "node_modules/@patternfly/react-icons": { - "version": "4.93.3", - "resolved": "https://registry.npmjs.org/@patternfly/react-icons/-/react-icons-4.93.3.tgz", - "integrity": "sha512-OIEeTc4Noi9XOIRF3OB3sz9dRnxr1h4eNIzIeZwRd8xXWCQxYcrllxPV98F3+RpL4ZCH2QWb/2gG4mHrVyX+0A==", + "version": "4.93.6", + "resolved": "https://registry.npmjs.org/@patternfly/react-icons/-/react-icons-4.93.6.tgz", + "integrity": "sha512-ZrXegc/81oiuTIeWvoHb3nG/eZODbB4rYmekBEsrbiysyO7m/sUFoi/RLvgFINrRoh6YCJqL5fj06Jg6d7jX1g==", "peerDependencies": { "react": "^16.8 || ^17 || ^18", "react-dom": "^16.8 || ^17 || ^18" @@ -32576,9 +32576,9 @@ } }, "@patternfly/react-icons": { - "version": "4.93.3", - "resolved": "https://registry.npmjs.org/@patternfly/react-icons/-/react-icons-4.93.3.tgz", - "integrity": "sha512-OIEeTc4Noi9XOIRF3OB3sz9dRnxr1h4eNIzIeZwRd8xXWCQxYcrllxPV98F3+RpL4ZCH2QWb/2gG4mHrVyX+0A==", + "version": "4.93.6", + "resolved": "https://registry.npmjs.org/@patternfly/react-icons/-/react-icons-4.93.6.tgz", + "integrity": "sha512-ZrXegc/81oiuTIeWvoHb3nG/eZODbB4rYmekBEsrbiysyO7m/sUFoi/RLvgFINrRoh6YCJqL5fj06Jg6d7jX1g==", "requires": {} }, "@patternfly/react-styles": { diff --git a/package.json b/package.json index ae1058584..512f7c0f9 100644 --- a/package.json +++ b/package.json @@ -7,7 +7,7 @@ "@data-driven-forms/pf4-component-mapper": "^3.11.3", "@data-driven-forms/react-form-renderer": "^3.16.5", "@patternfly/react-core": "^4.267.6", - "@patternfly/react-icons": "^4.93.3", + "@patternfly/react-icons": "^4.93.6", "@patternfly/react-table": "^4.112.6", "@redhat-cloud-services/frontend-components": "3.9.27", "@redhat-cloud-services/frontend-components-notifications": "^3.2.12",