Skip to content

fix(ui): preserve TagLabel server fields when saving tags (#28038) - #28105

Merged
siddhant1 merged 2 commits into
mainfrom
sid/fix-28038-tag-applied-by
May 15, 2026
Merged

fix(ui): preserve TagLabel server fields when saving tags (#28038)#28105
siddhant1 merged 2 commits into
mainfrom
sid/fix-28038-tag-applied-by

chore(ui): drop unused TagLabel/EntityTags imports from TagsContainerV2

5f0a456
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 14, 2026 in 7m 27s

Annotations

Check warning on line 22 in src/utils/Lineage/Layout/ELKUtil/ELKUtil.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=AZ4lQOG5j1xLa1vtLp_A&open=AZ4lQOG5j1xLa1vtLp_A&pullRequest=28105

Check failure on line 107 in src/components/common/atoms/asyncTreeSelect/useTreeSelection.tsx

See this annotation in the file changed.

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

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

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

Check warning on line 481 in src/components/AuditLog/AuditLogList.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=AZ4lQK0Mj1xLa1vtLp60&open=AZ4lQK0Mj1xLa1vtLp60&pullRequest=28105

Check warning on line 90 in src/pages/LineageConfigPage/LineageConfigPage.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since the receiver accepts the original type of the expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4lQN-Aj1xLa1vtLp-2&open=AZ4lQN-Aj1xLa1vtLp-2&pullRequest=28105

Check warning on line 748 in src/pages/TableDetailsPageV1/TableDetailsPageV1.tsx

See this annotation in the file changed.

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

Unexpected negated condition.

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

Check warning on line 310 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

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

Prefer `Number.parseInt` over `parseInt`.

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

Check warning on line 97 in src/components/DataAssets/DomainLabelV2/DomainLabelV2.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since the receiver accepts the original type of the expression.

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

Check failure on line 571 in src/components/Auth/AuthProviders/AuthProvider.tsx

See this annotation in the file changed.

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

Refactor this code to not nest functions more than 4 levels deep.

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

Check warning on line 295 in src/utils/CommonUtils.tsx

See this annotation in the file changed.

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

Unexpected negated condition.

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

Check failure on line 1777 in src/pages/ColumnBulkOperations/ColumnGrid/ColumnGrid.component.tsx

See this annotation in the file changed.

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

Refactor this code to not nest functions more than 4 levels deep.

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

Check warning on line 96 in src/components/Settings/Team/TeamDetails/TeamsHeaderSection/TeamsHeadingLabel.component.tsx

See this annotation in the file changed.

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

Avoid non-native interactive elements. If using native HTML is not possible, add an appropriate role and support for tabbing, mouse, keyboard, and touch inputs to an interactive content element.

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

Check warning on line 237 in src/components/ExploreV1/ExploreV1.component.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=AZ4lQLdxj1xLa1vtLp71&open=AZ4lQLdxj1xLa1vtLp71&pullRequest=28105

Check failure on line 2271 in src/pages/ColumnBulkOperations/ColumnGrid/ColumnGrid.component.tsx

See this annotation in the file changed.

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

Refactor this code to not nest functions more than 4 levels deep.

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

Check warning on line 40 in src/components/WorkflowDefinitions/Workflows/Nodes/GatewayNode/GatewayNode.tsx

See this annotation in the file changed.

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

Avoid non-native interactive elements. If using native HTML is not possible, add an appropriate role and support for tabbing, mouse, keyboard, and touch inputs to an interactive content element.

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

Check warning on line 472 in src/components/Settings/Applications/AppRunsHistory/AppRunsHistory.component.tsx

See this annotation in the file changed.

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

Unexpected negated condition.

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

Check warning on line 56 in src/constants/Learning.constants.ts

See this annotation in the file changed.

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

Use `.includes()` instead of `.some()` when checking value existence.

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

Check failure on line 464 in src/components/DataQuality/AddDataQualityTest/components/ParameterForm.tsx

See this annotation in the file changed.

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

Refactor this code to not nest functions more than 4 levels deep.

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

Check warning on line 114 in src/constants/Schedular.constants.ts

See this annotation in the file changed.

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

Simplify this regular expression to reduce its complexity from 51 to the 20 allowed.

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

Check warning on line 50 in src/components/common/MuiDrawer/MuiDrawer.tsx

See this annotation in the file changed.

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

'PaperProps' is deprecated.

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

Check warning on line 439 in src/components/Customization/CustomizeTabWidget/CustomizeTabWidget.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since the receiver accepts the original type of the expression.

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

Check failure on line 245 in src/components/common/Form/JSONSchema/JsonSchemaWidgets/LdapRoleMappingWidget/LdapRoleMappingWidget.tsx

See this annotation in the file changed.

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

Refactor this code to not nest functions more than 4 levels deep.

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

Check failure on line 269 in src/components/MyData/Widgets/KPIWidget/KPIWidget.component.tsx

See this annotation in the file changed.

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

Provide a compare function to avoid sorting elements alphabetically.

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

Check warning on line 365 in src/components/common/CoverImageUpload/MUICoverImageUpload.tsx

See this annotation in the file changed.

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

Prefer `globalThis` over `window`.

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

Check warning on line 70 in src/pages/RolesPage/RolesDetailPage/RolesDetailPage.tsx

See this annotation in the file changed.

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

useState call is not destructured into value + setter pair

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

Check warning on line 59 in src/utils/RouterUtils.ts

See this annotation in the file changed.

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

Prefer `.some(…)` over `.find(…)`.

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