yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2025-08-07 16:48:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix #1236 Version Summary for owner change is missing data. (#1312)

Browse Source
This commit is contained in:
Sachin Chaurasiya 2021-11-22 12:43:50 +05:30 committed by GitHub
parent f6e31019fe
commit fbbabb7349
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
catalog-rest-service/src/main/resources/ui/src/utils/EntityVersionUtils.tsx
View File

@ -169,6 +169,8 @@ export const summaryFormatter = (v: FieldChange) => {
return `columns ${value?.map((val: any) => val?.name).join(', ')}`; return `columns ${value?.map((val: any) => val?.name).join(', ')}`;
} else if (v.name === 'tags' || v.name?.endsWith('tags')) { } else if (v.name === 'tags' || v.name?.endsWith('tags')) {
return `tags ${value?.map((val: any) => val?.tagFQN)?.join(', ')}`; return `tags ${value?.map((val: any) => val?.tagFQN)?.join(', ')}`;
} else if (v.name === 'owner') {
return `${v.name} ${value.name}`;
} else { } else {
return v.name; return v.name;
} }
@ -183,17 +185,17 @@ export const getSummary = (changeDescription: ChangeDescription) => {
<Fragment> <Fragment>
{fieldsAdded?.length > 0 ? ( {fieldsAdded?.length > 0 ? (
<p className="tw-mb-2"> <p className="tw-mb-2">
{fieldsAdded?.map(summaryFormatter)} has been added {fieldsAdded?.map(summaryFormatter).join(', ')} has been added
</p> </p>
) : null} ) : null}
{fieldsUpdated?.length ? ( {fieldsUpdated?.length ? (
<p className="tw-mb-2"> <p className="tw-mb-2">
{fieldsUpdated?.map(summaryFormatter)} has been updated {fieldsUpdated?.map(summaryFormatter).join(', ')} has been updated
</p> </p>
) : null} ) : null}
{fieldsDeleted?.length ? ( {fieldsDeleted?.length ? (
<p className="tw-mb-2"> <p className="tw-mb-2">
{fieldsDeleted?.map(summaryFormatter)} has been deleted {fieldsDeleted?.map(summaryFormatter).join(', ')} has been deleted
</p> </p>
) : null} ) : null}
</Fragment> </Fragment>