Skip to content

Commit

Permalink
add reference to other key values
Browse files Browse the repository at this point in the history
Signed-off-by: Ben Dye <[email protected]>
  • Loading branch information
B-T-D committed Sep 11, 2024
1 parent fb1762f commit bf12883
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,11 @@ export const mapStateToProps = (state: GlobalState) => {
const ownerObj = state.tableMetadata.tableOwners.owners;
const items = Object.keys(ownerObj).reduce((obj, ownerId) => {
// eslint-disable-next-line @typescript-eslint/naming-convention
const { profile_url, user_id, display_name } = ownerObj[ownerId];
const { profile_url, user_id, display_name, other_key_values } =
ownerObj[ownerId];
let profileLink = profile_url;
let isExternalLink = true;
let additionalOwnerInfo = other_key_values;

Check failure on line 26 in frontend/amundsen_application/static/js/pages/TableDetailPage/TableOwnerEditor/index.tsx

View workflow job for this annotation

GitHub Actions / test-unit-frontend (12.x)

'additionalOwnerInfo' is never reassigned. Use 'const' instead

if (indexUsersEnabled()) {
isExternalLink = false;
Expand All @@ -32,6 +34,7 @@ export const mapStateToProps = (state: GlobalState) => {
label: display_name,
link: profileLink,
isExternal: isExternalLink,
additionalOwnerInfo: additionalOwnerInfo,

Check failure on line 37 in frontend/amundsen_application/static/js/pages/TableDetailPage/TableOwnerEditor/index.tsx

View workflow job for this annotation

GitHub Actions / test-unit-frontend (12.x)

Expected property shorthand
};

return obj;
Expand Down

0 comments on commit bf12883

Please sign in to comment.