Update opensight from main #4159
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
An instance variable stores the value directly and doesn't cause re-renders if it is changed. Variables returned from this hooks are comparable to instance variables for class components.
The dependency review workflow is able to write a summary message to the PR if it is allowed to write to the workflow.
The useShallowEqualSelector hooks allows to avoid re-renders if an object is selected from the redux store but its value(s) didn't change. With the standard selector which uses `===` comparison even updating an object's value to the same value will cause a re-render (because a new state object is created). This pattern can be found at https://react-redux.js.org/api/hooks#recipe-useshallowequalselector
The new usePageFilter hook allows to get the current applied filter of a page from the redux store.
Update the usePageFilter hook to add additional functions to change, reset and remove a page filter.
The selectors where invented for usage with mapStateToProps therefore they return objects at the moment. To avoid unnecessary re-renders the returned objects need to be compared with shallow equal.
At the end FilterProvider should be replaced with usePageFilter completely.
Allow for better reading flow where variables and hooks are defined at the top.
* Add: CVSS 4.0 Calculator * Add: test for CvssV4Point0Calculator * Apply: review comments * Add: German translations
Add a new hooks that allows to get the previous value after the value is changed. For example this hook can be used to check whether a filter has changed.
CVSS 4.0 metrics can now be displayed in CVE details.
for translation and consistency with the calculator.
Support CVSS 4.0 fields in CVEs.
The hook can be used to run a function after a specific amount of time for example for doing a reload of data.
* Change: withIconSize HOC to hook * add tests * improve test coverage * remove snapshot test
Bumps [braces](https://github.com/micromatch/braces) from 3.0.2 to 3.0.3. - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](micromatch/braces@3.0.2...3.0.3) --- updated-dependencies: - dependency-name: braces dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
…s-3.0.3 Bump braces from 3.0.2 to 3.0.3
The new useReload hook is based in useTiming. It calls a timing function before every reload to calculate the timeout before the reload. This timing function gets a isVisible argument passed. Using the argument the timing function can decide to extend the timeout when the current browser window is not visible.
The hook determines the sort field and direction of a filter and allows to change both via a returned function. The hook can be used to implement the filter changes when clicking on the different header columns of a entities list.
The hook returns functions the update a filter for getting the next, previous, first and last page for a list of entities.
The useSelection hook implements the entity selection at a entities list table. It is possible to select/deselect specific entities from the list/table, to select all entities displayed at the page or all entities for the current filter (filter without rows value applied).
Conventional Commits Report
🚀 Conventional commits found. |
fix-CI-step--upload-coverage
Co-authored-by: Ahmed <[email protected]>
…-setting fix: default filter selection in details page
Removes [path-to-regexp](https://github.com/pillarjs/path-to-regexp). It's no longer used after updating ancestor dependency [react-router-dom](https://github.com/remix-run/react-router/tree/HEAD/packages/react-router-dom). These dependencies need to be updated together. Removes `path-to-regexp` Updates `react-router-dom` from 5.3.4 to 6.26.2 - [Release notes](https://github.com/remix-run/react-router/releases) - [Changelog](https://github.com/remix-run/react-router/blob/main/packages/react-router-dom/CHANGELOG.md) - [Commits](https://github.com/remix-run/react-router/commits/[email protected]/packages/react-router-dom) --- updated-dependencies: - dependency-name: path-to-regexp dependency-type: indirect - dependency-name: react-router-dom dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
…-be58494012 Bump path-to-regexp and react-router-dom
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.4.3 to 5.4.6. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v5.4.6/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v5.4.6/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]>
…5.4.6 Bump vite from 5.4.3 to 5.4.6
daniele-mng
force-pushed
the
update-from-main
branch
from
5b8b6c4
to
60f2739
Compare
a-h-abdelsalam
previously approved these changes
Sep 24, 2024
a-h-abdelsalam
approved these changes
Sep 24, 2024
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## opensight-ui #4159 +/- ##
================================================
- Coverage 72.32% 63.34% -8.99%
================================================
Files 1068 1107 +39
Lines 122474 103441 -19033
Branches 6086 6421 +335
================================================
- Hits 88584 65522 -23062
- Misses 33859 37753 +3894
- Partials 31 166 +135 ☔ View full report in Codecov by Sentry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What
Why
References
Checklist