Skip to content

chore(UI): upgrade react-router-dom v6 → v7 and fix column panel race condition - #30580

Open
Rohit0301 wants to merge 4 commits into
mainfrom
upgrade-react-router-dom-v7
Open

chore(UI): upgrade react-router-dom v6 → v7 and fix column panel race condition#30580
Rohit0301 wants to merge 4 commits into
mainfrom
upgrade-react-router-dom-v7

Merge branch 'main' into upgrade-react-router-dom-v7

ac7e730
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded Jul 29, 2026 in 9m 2s

Annotations

Check warning on line 52 in src/utils/TagClassBase.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Make this public static property readonly.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow4GqV3huzmeJ2dJr&open=AZ-ow4GqV3huzmeJ2dJr&pullRequest=30580

Check warning on line 92 in src/components/common/PopOverCard/EntityPopOverCard.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow1f4V3huzmeJ2c9r&open=AZ-ow1f4V3huzmeJ2c9r&pullRequest=30580

Check warning on line 80 in src/utils/EdgeStyleUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Function 'computeEdgeStyle' has too many parameters (10). Maximum allowed is 7.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow4ILV3huzmeJ2dJv&open=AZ-ow4ILV3huzmeJ2dJv&pullRequest=30580

Check warning on line 155 in src/components/DataAssets/DataAssetsHeader/DataAssetsHeader.interface.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Consider removing 'undefined' type or '?' specifier, one of them is redundant.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow3HPV3huzmeJ2dD9&open=AZ-ow3HPV3huzmeJ2dD9&pullRequest=30580

Check warning on line 76 in src/constants/regex.constants.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Simplify this regular expression to reduce its runtime, as it has super-linear performance due to backtracking.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow5FiV3huzmeJ2dMv&open=AZ-ow5FiV3huzmeJ2dMv&pullRequest=30580

Check warning on line 41 in src/components/common/EntityImport/CsvCellPreview/CsvCellPreview.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

`AVATAR_COLUMNS` should be a `Set`, and use `AVATAR_COLUMNS.has()` to check existence or non-existence.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow1ShV3huzmeJ2c87&open=AZ-ow1ShV3huzmeJ2c87&pullRequest=30580

Check warning on line 366 in src/components/SettingsSso/SSOGroupedFieldTemplate/SSOGroupedFieldTemplate.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Do not use Array index in keys

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow29nV3huzmeJ2dDf&open=AZ-ow29nV3huzmeJ2dDf&pullRequest=30580

Check warning on line 104 in src/utils/QueryBuilderElasticsearchFormatUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This case's code block is the same as the block for the case on line 91.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow4EgV3huzmeJ2dJG&open=AZ-ow4EgV3huzmeJ2dJG&pullRequest=30580

Check failure on line 41 in src/components/BlockEditor/Extensions/diff-view.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `getAttribute(…)`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow0ttV3huzmeJ2c7D&open=AZ-ow0ttV3huzmeJ2c7D&pullRequest=30580

Check warning on line 85 in src/components/SearchSettings/GlobalSettingsItem/GlobalSettingsItem.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.parseFloat` over `parseFloat`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow3bOV3huzmeJ2dFt&open=AZ-ow3bOV3huzmeJ2dFt&pullRequest=30580

Check warning on line 543 in src/components/DataContract/ContractSecurityFormTab/ContractSecurityFormTab.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow3N8V3huzmeJ2dEV&open=AZ-ow3N8V3huzmeJ2dEV&pullRequest=30580

Check warning on line 79 in src/constants/regex.constants.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Simplify this regular expression to reduce its runtime, as it has super-linear performance due to backtracking.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow5FiV3huzmeJ2dMw&open=AZ-ow5FiV3huzmeJ2dMw&pullRequest=30580

Check warning on line 47 in src/components/MyData/Widgets/KPIWidget/KPILegend/KPILegend.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Move this component definition out of the parent component and pass data as props.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow15sV3huzmeJ2c_Q&open=AZ-ow15sV3huzmeJ2c_Q&pullRequest=30580

Check failure on line 122 in src/components/OntologyExplorer/utils/graphConfig.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow3QkV3huzmeJ2dEl&open=AZ-ow3QkV3huzmeJ2dEl&pullRequest=30580

Check warning on line 304 in src/pages/TagPage/TagPage.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow48HV3huzmeJ2dMT&open=AZ-ow48HV3huzmeJ2dMT&pullRequest=30580

Check warning on line 36 in src/rest/searchAPI.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use Array.isArray() instead of isArray() from lodash.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow5DGV3huzmeJ2dMg&open=AZ-ow5DGV3huzmeJ2dMg&pullRequest=30580

Check warning on line 122 in src/components/Settings/Users/UsersProfile/UserPermissions/UserPermissions.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Do not use Array index in keys

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow2VgV3huzmeJ2dBC&open=AZ-ow2VgV3huzmeJ2dBC&pullRequest=30580

Check warning on line 251 in src/components/common/UserTeamSelectableList/UserTeamSelectableList.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow1c8V3huzmeJ2c9k&open=AZ-ow1c8V3huzmeJ2c9k&pullRequest=30580

Check warning on line 141 in src/pages/LearningResourcesPage/LearningResourceForm.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use the "RegExp.exec()" method instead.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow4vqV3huzmeJ2dLh&open=AZ-ow4vqV3huzmeJ2dLh&pullRequest=30580

Check failure on line 224 in src/rest/columnAPI.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow5AMV3huzmeJ2dMZ&open=AZ-ow5AMV3huzmeJ2dMZ&pullRequest=30580

Check warning on line 121 in src/components/MyData/CustomizableComponents/CustomiseLandingPageHeader/RecentlyViewedCarousel.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use <input type="button">, <input type="image">, <input type="reset">, <input type="submit">, or <button> instead of the "button" role to ensure accessibility across all devices.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow11xV3huzmeJ2c_B&open=AZ-ow11xV3huzmeJ2c_B&pullRequest=30580

Check warning on line 115 in src/components/Glossary/GlossaryTerms/tabs/WorkFlowTab/WorkflowHistory.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Move this array "reverse" operation to a separate statement or replace it with "toReversed".

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow2gHV3huzmeJ2dBu&open=AZ-ow2gHV3huzmeJ2dBu&pullRequest=30580

Check warning on line 67 in src/components/Alerts/DestinationFormItem/TeamAndUserSelectItem/TeamAndUserSelectItem.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow3C6V3huzmeJ2dDz&open=AZ-ow3C6V3huzmeJ2dDz&pullRequest=30580

Check warning on line 523 in src/components/Domain/AddDomainForm/AddDomainForm.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Consider removing 'undefined' type or '?' specifier, one of them is redundant.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow1ztV3huzmeJ2c-6&open=AZ-ow1ztV3huzmeJ2c-6&pullRequest=30580

Check warning on line 244 in src/rest/suggestionsAPI.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-ow5BcV3huzmeJ2dMc&open=AZ-ow5BcV3huzmeJ2dMc&pullRequest=30580